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

add build test #48

Merged
merged 9 commits into from
Jun 18, 2024
Merged

add build test #48

merged 9 commits into from
Jun 18, 2024

Conversation

djay
Copy link
Member

@djay djay commented Jun 13, 2024

No description provided.

@djay
Copy link
Member Author

djay commented Jun 13, 2024

@MAX-786 @JeffersonBledsoe last deploy got a failure

2.414  ERR_PNPM_OUTDATED_LOCKFILE  Cannot install with "frozen-lockfile" because pnpm-lock.yaml is not up to date with package.json

I want to put in a test that ensures the build is going to work at least but not really sure how to do this. can someone fix this?

@MAX-786
Copy link
Member

MAX-786 commented Jun 13, 2024

We can add this in the test file to check for lock-file and update lockfile according to package.json:

    - name: Check pnpm lockfile consistency
        run: |
          pnpm install --lockfile-only
          

And i don't know why its showing mrs.developer.json is missing.
@JeffersonBledsoe Will this be okay to do so?

@djay djay merged commit 81f5e05 into main Jun 18, 2024
1 check passed
@djay djay deleted the add_build_test branch June 18, 2024 03:44
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