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: remove upload of fragments to github #8400

Merged
merged 2 commits into from
Jan 11, 2024

Conversation

Saschl
Copy link
Member

@Saschl Saschl commented Jan 10, 2024

Installer uses cf and we upload the full package to github as an alternative for the installer.

Summary of Changes

Removing the upload of fragments to github as I could not find/remember any usage currently

Screenshots (if necessary)

References

Additional context

Discord username (if different from GitHub):

Testing instructions

How to download the PR for QA

Every new commit to this PR will cause a new A32NX artifact to be created, built, and uploaded.

  1. Make sure you are signed in to GitHub
  2. Click on the Checks tab on the PR
  3. On the left side, click on the bottom PR tab
  4. Click on the A32NX download link at the bottom of the page

Installer uses cf and we upload the full package to github as an alternative for installing
@github-actions github-actions bot added this to 🟡 Code Review: Ready for Review in Quality Assurance Jan 10, 2024
Quality Assurance automation moved this from 🟡 Code Review: Ready for Review to 🟣 QA Team Review: Ready to Test Jan 10, 2024
@Benjozork Benjozork changed the title fix: remove upload of fragments to github remove upload of fragments to github Jan 10, 2024
@Benjozork Benjozork changed the title remove upload of fragments to github fix: remove upload of fragments to github Jan 10, 2024
@Saschl
Copy link
Member Author

Saschl commented Jan 11, 2024

Merging, in case we still want GitHub as fallback, we can restore the changes and make the necessary adaptions.

@Saschl Saschl enabled auto-merge (squash) January 11, 2024 06:34
@Saschl Saschl merged commit 86560d7 into flybywiresim:master Jan 11, 2024
7 checks passed
Quality Assurance automation moved this from 🟣 QA Team Review: Ready to Test to ✔️ Done Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants