Skip to content

Remove upload --no-pack option from README#18

Open
bchassoul wants to merge 2 commits intomainfrom
bchassoul/readme
Open

Remove upload --no-pack option from README#18
bchassoul wants to merge 2 commits intomainfrom
bchassoul/readme

Conversation

@bchassoul
Copy link
Member

@bchassoul bchassoul commented Mar 2, 2025

While testing the latest grisp_connect I realized we no longer support the --no-pack option in upload

> rebar3 as grisp-io upload --no-pack
===> Invalid option --no-pack on task upload

Feel free to close this PR in case that's a bug and not a feature 🥲

@bchassoul bchassoul requested a review from GwendalLaurent March 2, 2025 18:50
This command has 2 options:
This command has 1 optional option:
- `--force` or `-f`: This option will force an overwritting of the local and remote files of a given project and release
- `--no-pack` or `-p`: With this option enabled, the command won't run internally the command `rebar3 grisp pack`
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe replace it but the refresh option: --refresh -r: Force software package building even if it already exists

Copy link
Member Author

Choose a reason for hiding this comment

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

ah true! done :)

@bchassoul bchassoul requested a review from sylane March 3, 2025 15:59
@bchassoul
Copy link
Member Author

@GwendalLaurent @sylane Do we know why the CI fails? should I merge?

@GwendalLaurent
Copy link
Member

@GwendalLaurent @sylane Do we know why the CI fails? should I merge?

Some of our CI has been broken after we changed the way we initialize mnesia. In this case eresu:install/1 has been removed from eresu in this commit. We need to udpate the tests of some of our repos.

I had the same issue in seabac for this PR

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.

3 participants