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

[misc] release process refactor #360

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

[misc] release process refactor #360

wants to merge 22 commits into from

Conversation

connorhu
Copy link
Collaborator

@connorhu connorhu commented Mar 27, 2024 •
edited
Loading

The aim of PR is to automate the processes needed to create a version:

  • change the version in the sfCoreAutoload file
  • sets changelog header
  • creates the tag, tags the current source
  • pushes all changes
  • GHA job extracts changes to the tag from the changelog and makes a new release. similar to this: sebastianbergmann/foal@b0e3537

closes #354

@connorhu connorhu force-pushed the fix/refactor-release-script branch from 6cab55e to ca10bb7 Compare March 27, 2024 09:45
@connorhu
Copy link
Collaborator Author

  • I would separate the scripts I wrote from the data/bin library (which I don't like anyway). They are maintenance scripts. The place where I put them now is not quite right for them, I should find a better place for them.
  • I minimized the use of 3rd party libraries. The lime_output class could be used for output coloring, but if we ever get to the point where phpunit takes the place of lime I'd rather not use it and use minimal tools.

@connorhu connorhu marked this pull request as ready for review March 27, 2024 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

update SYMFONY_VERSION constant accordingly
1 participant
@connorhu