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 devshell #3

Merged
merged 4 commits into from
Feb 24, 2024
Merged

fix devshell #3

merged 4 commits into from
Feb 24, 2024

Conversation

yanganto
Copy link

  • MSRV should be the default, or the developer will make the issue as I mentioned before.
  • Update should be in a different CI process but not the shell
  • Lock project this is required to have a reproducible result

storopoli

This comment was marked as outdated.

@storopoli storopoli self-requested a review February 24, 2024 10:14
@storopoli
Copy link
Owner

We have some CI failures. You wanna try fixing them?

@yanganto yanganto marked this pull request as draft February 24, 2024 10:37
@yanganto yanganto marked this pull request as ready for review February 24, 2024 10:58
@yanganto yanganto marked this pull request as draft February 24, 2024 11:03
@yanganto
Copy link
Author

yanganto commented Feb 24, 2024

Give me one minute, I am fixing the sign check.

@yanganto yanganto force-pushed the msrv-lock branch 2 times, most recently from 3085dc6 to 6f382d4 Compare February 24, 2024 11:29
@yanganto yanganto marked this pull request as ready for review February 24, 2024 11:48
@yanganto
Copy link
Author

Hi @storopoli
All green now. Thanks for accept this idea.

Copy link
Owner

@storopoli storopoli left a comment

Choose a reason for hiding this comment

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

Thanks for that.
Ok buddy looks like you and me will be the Cargo.lock babysitters 😄

@storopoli storopoli merged commit ed85ca6 into storopoli:js/nix-simple Feb 24, 2024
13 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
2 participants