Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Package.json to Allow Windows Install Without WSL #93

Merged

Conversation

JMielbrecht
Copy link
Contributor

PR Author's Note

Fixed dependencies in package.json to enable npm install on windows without the following EBADPLATFORM error preventing the installation:

npm ERR! code EBADPLATFORM
npm ERR! notsup Unsupported platform for @napi-rs/[email protected]: wanted {"os":"linux","arch":"x64"} (current: {"os":"win32","arch":"x64"})
npm ERR! notsup Valid OS:    linux
npm ERR! notsup Valid Arch:  x64
npm ERR! notsup Actual OS:   win32
npm ERR! notsup Actual Arch: x64

Credit to @PatrickLarocque in his PR #90 for coming up with the fix; I've implemented it here as a standalone.

Checklist

  • Philosophical or literary contribution (docs). Leave unchecked for
    code contribution.

    • IMPLIED CONSENT By opening this pull request and contributing
      philosophical or literary content, I accept that my writing is submitted
      under the
      ATTRIBUTION-NONCOMMERCIAL-SHAREALIKE 4.0 INTERNATIONAL,
      which:

      • Prohibits commercial reuse of the content.
      • Allows sharing, remixing, and building upon the material as long as
        attribution is given.

      I understand that my writing may be modified, remixed, and built upon by
      others within the systemphil/sphil or sPhil project, in accordance
      with the license terms, indefinitely. See
      legal code.

    • REQUIRED I have followed the
      formatting guidelines
      and verified there are no formatting bugs.
      Try markdown preview here.

    • REQUIRED I have followed the
      Chicago author-date style.

    • REQUIRED I have added or verified metadata title, description, and
      contributors at the very top of the file followed by a ## title
      heading. Additionally, I have ensured isArticle is set to true.
      Example:

      ---
      title: The Immediate Difference Between Pure Being and Pure Nothing
      description:
          Learn about the difference between being and nothing in Hegel's
          Science of Logic.
      isArticle: true
      authors: Jerry Maguire (2024)
      editors: Steve Stevenson (2023), Karen Hansen (2022)
      contributors:
      ---
      
      ## My Article Title
      Further information

      I have signed the document with my name/username under either as
      Authors, Editors or Contributors.

      Use Authors if you have created and substantially added content.
      Use Editor if you have made substantial edits or review.
      Use Contributor if you have made minor edits, reviews or
      contributions.
      If you've done multiple, pick the most weighted: Author > Editor >
      Contributor.
      If you prefer to remain anonymous, that's fine too, but note that a
      record of your contributions based on your GitHub username will exist
      here in the codebase.

    • REQUIRED I have ensured that the
      project's central bibliography
      contains the necessary bibliographical details for the citations I have
      used.

    • Optional My article is a stub or I want to actively encourage
      contribution, I've added the Stub component to the bottom of my content
      or where relevant:

      import Stub from "@/components/Stub";
      
      <Stub />;
  • If Docs contribution is unchecked: Code contribution
    (Apache version 2 license)

    All code apart of what is inside src/pages/** (excluding
    /contributing/**, _app.mdx, _document.tsx, _meta.json,
    acknowledgements.mdx, index.mdx, privacy.mdx, team.mdx, terms.mdx)
    is subject to Apache version 2 license. Basically, anything outside of
    content, literature, philosophy.

@JMielbrecht JMielbrecht marked this pull request as ready for review October 9, 2024 04:57
Copy link
Member

@Firgrep Firgrep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good stuff! 😃

@Firgrep Firgrep merged commit 46a1bd8 into systemphil:main Oct 9, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants