-
Notifications
You must be signed in to change notification settings - Fork 89
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
Remove compiled setup scripts from Git, use CI instead #462
Comments
From the issue title do you mean that these amber scripts should be CI updated? I agree but I think that we just need to recompile them at every new release inside the repository. |
Committing obscure script that installs the software without approves is sus ngl ;) We should compile add only via CI or/and check them in CI if they yield the same result |
@Mte90, I don't see a point in keeping compiled scripts in the repo. That can only lead to problems. |
I'm all in. This issue should have "sub issues" to update the install snippet on the website and documentation |
The install/uninstall scripts can't be in Amber, otherwise users how can they use them? |
Again, the idea is that the CI will compile Amber to Bash, and include it into the release. |
So the Bash script will be in the release page? I am asking for this because usually that file is placed in the repository itself. |
Yes, it will be in the releases. Placing it in the repo itself may make total sense for other projects, but only because they write it in Bash by hand. Our Bash scripts are just an artifact from the Amber files that are in the repo. |
It is a pattern that I never saw honestly, could make sense but in the past we updated the installer also without doing a new release. |
We have the CI but we will remove the files from the repo at the next release so we can update the docs. |
Having both the source Amber files and the generated Bash files checked out in Git leads to inconsistency. For example see PR #457, which updates generated Bash files, as they were outdated.
Instead, lets set up a CI pipeline which will publish the generated Bash scripts at each release, and reference them in the installation instructions.
The text was updated successfully, but these errors were encountered: