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

build: updated dependencies and improve title and version in content manager #6461

Merged
merged 1 commit into from
Jan 1, 2022
Merged

build: updated dependencies and improve title and version in content manager #6461

merged 1 commit into from
Jan 1, 2022

Conversation

aguther
Copy link
Contributor

@aguther aguther commented Jan 1, 2022

Summary of Changes

This PR introduces an improved title and versioning in the content manager of the sim:

  • Version
    • It will show the short git commit hash along with the version
  • Title
    • It will show "A32NX (STABLE)" for stable version
    • It will show "A32NX (DEVELOPMENT)" for development version
    • It will show "A32NX (BRANCH xy)" for any other build

Screenshots (if necessary)

image

Testing instructions

  • test if this package can be loaded successfully and the git commit hash is shown and the title as spec above

Remark: not all kinds of editions can be tested for the PR. This will take place when we update the development version with this PR and when the next stable version is released.

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

@aguther aguther merged commit 6f51411 into flybywiresim:master Jan 1, 2022
@aguther aguther deleted the feat/title-version-in-content-manager branch January 1, 2022 23:41
@2hwk 2hwk added this to the v0.8.0 milestone Apr 29, 2022
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