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

Add v3.18 to TestPackageAlpine #24972

Merged
merged 10 commits into from
May 29, 2023
Merged

Conversation

sonjek
Copy link
Contributor

@sonjek sonjek commented May 28, 2023

Add Alpine 3.18 to TestPackageAlpine

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label May 28, 2023
@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label May 28, 2023
@silverwind
Copy link
Member

Can you explain what this test does with these alpine versions?

@sonjek
Copy link
Contributor Author

sonjek commented May 29, 2023

This test verifies the ability to push a package for the specified version of Alpine, checks the correctness of filling in all the data, and performs package removal.

This test verifies the ability to push a package for the specified version of Alpine, checks the correctness of filling in all the data, and performs package removal. Alpine 3.18 is a last one stable version. Currently, Gitea is using Alpine version 3.18, so it is valid to also test it accordingly.

@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels May 29, 2023
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels May 29, 2023
@delvh delvh added reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. labels May 29, 2023
@silverwind silverwind enabled auto-merge (squash) May 29, 2023 10:41
@silverwind silverwind merged commit 970e1c9 into go-gitea:main May 29, 2023
@GiteaBot GiteaBot added this to the 1.20.0 milestone May 29, 2023
@GiteaBot GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label May 29, 2023
zjjhot added a commit to zjjhot/gitea that referenced this pull request May 30, 2023
* upstream/main:
  Remove unnecessary content on docs (go-gitea#24976)
  Upgrade snap to node 20 (go-gitea#24990)
  Display file mode for new file and file mode changes (go-gitea#24966)
  Enable `vue/html-closing-bracket-spacing` eslint rule (go-gitea#24987)
  Fix repo level project - edit column  (go-gitea#24982)
  Add v3.18 to TestPackageAlpine (go-gitea#24972)
  Fix admin config page error, use tests to cover the admin config and 500 error page (go-gitea#24965)
  Improve dropdown menus, remove inline styles (go-gitea#24954)
  Unify doc links to use paths relative to doc folder (go-gitea#24979)
  Rework button coloring, add focus and active colors (go-gitea#24507)
  Add PDF rendering via PDFObject (go-gitea#24086)
  Hide 'Mirror Settings' when unneeded, improve hints (go-gitea#24433)
  Consolidate the two review boxes into one (go-gitea#24738)
  API endpoint for changing/creating/deleting multiple files (go-gitea#24887)
  Repo list improvements, fix bold helper classes (go-gitea#24935)
  Add step start time to `ViewStepLog` (go-gitea#24980)
  Update from-binary.en-us.md (go-gitea#24975)
@KN4CK3R
Copy link
Member

KN4CK3R commented Jun 12, 2023

This PR is useless. The different versions in the test are only needed to... test different versions. The name could be dummy-123 and it would still work. It's not related to current Alpine version.

@silverwind
Copy link
Member

Okay, let's keep that in mind and not accept any further updates to that list.

@sonjek
Copy link
Contributor Author

sonjek commented Jun 17, 2023

Got it @KN4CK3R. Thank you for explanation.

@go-gitea go-gitea locked as resolved and limited conversation to collaborators Aug 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. type/testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants