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

WIP: deprecate stable and latest branches, rename to release #1582

Closed
wants to merge 4 commits into from

Conversation

nodiscc
Copy link
Member

@nodiscc nodiscc commented Oct 3, 2020

  • always set the target branch for update checks to release
  • update documentation (docker image tags, versioning, release procedure, README badges, migration)
  • doc: mention that the default-docker compose file uses the :latest image, update docker-compose documentation
  • rename the :master docker image tag to :latest
  • clarify method/variable names
  • fixes [Feature Request] Tag docker images with version and latest #1453

@nodiscc nodiscc added in progress documentation tools developer tools 3rd party interoperability with third-party platforms docker containers & cloud labels Oct 3, 2020
@nodiscc nodiscc self-assigned this Oct 3, 2020
@nodiscc nodiscc added this to the 0.12.0 milestone Oct 3, 2020
* Note: due to hardcoded URL and lack of dependency injection, this is not unit testable.
*
* FIXME! This needs to be removed when we switch to first digit major version
* instead of the second one since the versionning process will change.
*/
public function updateMethodCheckUpdateRemoteBranch()
Copy link
Member

Choose a reason for hiding this comment

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

I know this PR is a WiP, but I suggest that you let method as is - or remove it entirely - and create a new one in Shaarli\Updater\Updater: this one won't run again on instances that had already run it in the past.

@nodiscc nodiscc modified the milestones: 0.12.0, 0.13.0 Oct 12, 2020
 - always set the target branch for update checks to `release`
 - update documentation (docker image tags, versioning, release procedure, README badges, migration)
 - doc: mention that the default-docker compose file uses the `:latest` image, update docker-compose documentation
 - rename the `:master` docker image tag to `:latest`
 - clarify method/variable names
 - fixes shaarli#1453
@nodiscc nodiscc closed this Jun 26, 2021
@nodiscc nodiscc deleted the semver-release-latest branch June 26, 2021 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3rd party interoperability with third-party platforms docker containers & cloud documentation in progress tools developer tools
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request] Tag docker images with version and latest
2 participants