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.

Sign up for GitHub

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

Jump to bottom

Tag changes #13

Draft
wants to merge 65 commits into
base: master
Choose a base branch
from
Draft

Tag changes #13

wants to merge 65 commits into from

Conversation

WilliamWFLee
Copy link

No description provided.

@WilliamWFLee
Always use tags for indicating stages
e67c05f
@WilliamWFLee
Add deprecation warning to use_tags_for_deploy option
d6a5d9e
@WilliamWFLee
Add test for GitWrapper.update_tag_commit
6aeed3e
@WilliamWFLee
Clean up comment
23e9467
@WilliamWFLee
Simplify deploy feature specs setup
71e8492
@WilliamWFLee
Add test for release to call update_tag_commit when deploying
0eb0432
@WilliamWFLee
Require 'byebug' for tests
de14ff3
@WilliamWFLee
Add method to delete branches both locally and remotely
3aa3483
@WilliamWFLee
Add method on git wrapper to delete branches both locally and remotely
0c6a246
@WilliamWFLee
Change GitWrapper#delete_branches to accept an array of branch names
8207b14
@WilliamWFLee
Add #environments method to stages extractor to get a list of all dep…
7779af7 
…loyment environments and refactor environment matching to stages extractor
@WilliamWFLee
Delete legacy environment branches locally and on remote when deploying
9d52173
@WilliamWFLee
Improve use_tags_for_deploy option deprecation warnng
b703fad
@WilliamWFLee
Add Ruby 3.1, 3.2 and 3.3 to test workflow
2a3b593
@WilliamWFLee
Rename method for creating/updating tags and update implementation to…
1d80515 
… use git library methods
@WilliamWFLee
Add StagesExtractor#stages_for_environment to get deployable stages f…
8fe2721 
…or each environment
@WilliamWFLee
Update stages extractor so that either a valid stage or environment c…
9708588 
…an be passed to find the stages
@WilliamWFLee
Update Release.deploy! so that each deployment to a stage is done wit…
946d757 
…hin the library rather than using the deploy_all task for multi-user environments
@WilliamWFLee
Stub output helper methods instead of IO.puts to avoid silencing test…
d05f7c5 
… output
@WilliamWFLee
Move creation of deployment tags so that a tag for each stage is only…
3235f88 
… created when successfully deployed
@WilliamWFLee
Add use_timestamped_deploy_tags option for configuring use of new tag…
d83601b 
… style
@WilliamWFLee
Fix position of rescue clause
21576b3
@WilliamWFLee
Refactor release deployment logic to a deployer class
e9ba734
@WilliamWFLee
Add back method to create or update branch
3d9455e
@WilliamWFLee
Replace GitWrapper#tag with create_or_update_tag
dfd4268
@WilliamWFLee
Add old branch-style deployment behaviour back, using the use_timesta…
b008c32 
…mped_deploy_tags option to decide how to deploy
@WilliamWFLee
Add git wrapper method for fetching a list of tags for a given object…
2e7762e 
… reference
@WilliamWFLee
Update Release.create! so that it does not create a new release and t…
714670b 
…ag if the most recent commit is already a release commit
@WilliamWFLee
Fix specs for Release class
b324aad
@WilliamWFLee
Remove incorrect parameter for method
c5703bc
@ClearCarbon
Copy link

ensure you update the version number and update the readme, including instructions for transitioning to the new behaviour. most apps should not transition to the new behaviour until epiApps is updated to support it

@WilliamWFLee
Fix typo in release message
fea5e50
@WilliamWFLee
Remove unused method
a16341f
@WilliamWFLee
Use fully qualified ref names to avoid ambiguity between branches and…
6a7d003 
… tags with the same name
@WilliamWFLee
Fix tests
76bc8ae
@WilliamWFLee
Remove unused parameter
d6b6587
@WilliamWFLee
Use filter instead of find for finding local branches
54ed3d0
@WilliamWFLee
Update README with deployment tags information
f9baca3
@WilliamWFLee
Add epi_deploy:set_branch task for manually set the Capistrano :branc…
c174c8c 
…h variable on deployment
@WilliamWFLee
Update deployer to pass the branch reference as an environment variab…
7cdc07e 
…le to the capistrano task
@WilliamWFLee
Use require instead of require_relative in epi_deploy:set_branch task
ab67eeb
@WilliamWFLee
Use absolute path for requiring version.rb
2224f1b
@WilliamWFLee
Dereference tags to commit if a tag name is used as the reference
969c106
@WilliamWFLee
Remove setup class, and add latest_release_tag to AppVersion
e79b3e4
jamgregory
jamgregory reviewed Oct 3, 2024
View reviewed changes
README.md Outdated Show resolved Hide resolved
@WilliamWFLee
Make the default tag the latest release tag, rather than any tag
6f2a014
@WilliamWFLee
Add AppVersion.open method
de6b423
@WilliamWFLee
Fix AppVersion#initialize method
00afe71
@WilliamWFLee
Rename bump! method to bump
0dcb970
@WilliamWFLee
Fix saving of app version file
4fa3205
@WilliamWFLee
Fix release creation to create app version save file before creating …
c6a0b08 
…release
@WilliamWFLee
Fix tests
1f5a902
@WilliamWFLee
Update README to add instructions to include Capistrano tasks in Capf…
df6112c 
…ile for deployment tags
@WilliamWFLee
Add tests for the release setting the latest release tag in the versi…
5944114 
…on.rb when the release is created
@WilliamWFLee
Add tests for app version
16d82d4
@WilliamWFLee
Make regexes for version.rb more resilient to whitespace differences
d5bfc54
@WilliamWFLee
Do not define set_branch cap task if using default deployment behaviour
b19bdaf
@WilliamWFLee
Fix creation of branches for default behaviour
2a48c38
@WilliamWFLee
Add deploy- prefix to deployment tags to avoid conflict with tags tha…
789e954 
…t start with demo that are used for deployment purposes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jamgregory jamgregory jamgregory left review comments

@ClearCarbon ClearCarbon ClearCarbon approved these changes

Assignees

@WilliamWFLee WilliamWFLee

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@WilliamWFLee @ClearCarbon @jamgregory