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

Hotfix 4.1.7 #5556

Merged
merged 17 commits into from
Jan 10, 2022
Merged

Hotfix 4.1.7 #5556

merged 17 commits into from
Jan 10, 2022

Conversation

kenjis
Copy link
Member

@kenjis kenjis commented Jan 8, 2022

This is for hotfix release. Will be merged into master.

TODO

  • Update system/CodeIgniter.php with the new version number: const CI_VERSION = '4.x.x';
  • Update user_guide_src/source/conf.py with the new version = '4.x' (if applicable) and release = '4.x.x'
  • CHANGELOG.md
  • Update CHANGELOG.md release date
  • Set the date in user_guide_src/source/changelogs/{version}.rst to format Release Date: January 31, 2021
  • Create a new changelog for the next version at user_guide_src/source/changelogs/{next_version}.rst and add it to index.rst
  • If there are additional upgrade steps, create user_guide_src/source/installation/upgrade_{ver}.rst and add it to upgrading.rst

After merging

  • Merge this branch (or master) into develop

See User Guide, Website, Announcement
https://github.com/codeigniter4/CodeIgniter4/blob/develop/admin/RELEASE.md#user-guide

kenjis and others added 13 commits January 8, 2022 09:38
docs: add Security advisory in CHANGELOG.md
docs: minor addition to changelogs/v4.1.6.rst and upgrade_416.rst
Update modified Kint files in `ThirdParty`
fix: BaseConnection::getConnectDuration() number_format(): Passing null to parameter
docs: add upgrade instruction in upgrade_416.rst
Fix: Toolbar. ciDebugBar.showTab() context.
Replaced with FILTER_SANITIZE_FULL_SPECIAL_CHARS. Equivalent to calling htmlspecialchars() with ENT_QUOTES set.
Replaced with FILTER_SANITIZE_FULL_SPECIAL_CHARS. Equivalent to calling htmlspecialchars() with ENT_QUOTES set.
fix: replace deprecated FILTER_SANITIZE_STRING
@kenjis kenjis marked this pull request as draft January 8, 2022 02:38
@kenjis
Copy link
Member Author

kenjis commented Jan 8, 2022

What I did:

$ git fetch upstream
$ git checkout -b hotfix-4.1.7 upstream/master
$ git log -b develop --merges 

Pick up the merge commits.

67d00b3 #5553
a09f7d9 #5554
796acc8 #5546
a5d604d #5542
e0fd7d4 #5544
a8e271a #5536
347ac8e #5527
12877cd #5537
0755553 #5532

$ git cherry-pick -m 1 0755553a4259fbacc799395ca8a5546c5e979c45
$ git cherry-pick -m 1 12877cd742a00ec336bd54b599178e623e570c46
$ git cherry-pick -m 1 347ac8e092547dab7c3b575911bcef7c0fc95071
$ git cherry-pick -m 1 a8e271ae4666bb2dd3a51462264b16b07bbe4380
$ git cherry-pick -m 1 e0fd7d4cff0ef925b4ccfb876da1e8a64ece14e3
$ git cherry-pick -m 1 a5d604dc971c2fc383bcf12677bf9c1dd9bc4f23
$ git cherry-pick -m 1 796acc827b100238c466667c34861b4653d2ea07
$ git cherry-pick -m 1 a09f7d9f6814ea447ab4029ee99896ea5b0b069e
$ git cherry-pick -m 1 67d00b3ef762b25b59e8305f1d91bfd71a3b1eeb
$ git push upstream HEAD 

Create #5555 and merge.

Gereate CHANGELOG.md from hotfix-4.1.7 and develop branch.

Note: This work flow is not good for changelog auto-generation. Cherry-picked PRs is not included generated changelog for hotfix-4.1.7 branch.

@MGatner MGatner marked this pull request as ready for review January 10, 2022 01:31
@MGatner MGatner merged commit 99e0797 into master Jan 10, 2022
@kenjis kenjis deleted the hotfix-4.1.7 branch January 10, 2022 01:50
@kenjis
Copy link
Member Author

kenjis commented Jan 10, 2022

#5531 composer.json update should have been included the hotfix branch.
Because the previous fix commit in codeigniter4/framework was overwritten by the deployment script.

We have a GitHub Action that watches for a new release. It runs a deployment script:

- name: Deploy
run: ./source/.github/scripts/deploy-framework ${GITHUB_WORKSPACE}/source ${GITHUB_WORKSPACE}/framework ${GITHUB_REF##*/}

That script (https://github.com/codeigniter4/CodeIgniter4/blob/develop/.github/scripts/deploy-framework) copies all the files from the main repo to framework, but then overwrites a few files with the versions in admin/framework/

@kenjis
Copy link
Member Author

kenjis commented Jan 10, 2022

This work flow is not good for changelog auto-generation. Cherry-picked PRs is not included generated changelog for hotfix-4.1.7 branch.

And at the next release, the cherry-picked PRs included in v4.1.7 would probably be included again in auto-generated changelog for develop branch.

@MGatner
Copy link
Member

MGatner commented Jan 10, 2022

Since we reformat the changelog manually anyways I don't think that will be a problem, we just need to remember to pull any duplicates!

@kenjis
Copy link
Member Author

kenjis commented Jan 10, 2022

@MGatner When we need to release a hotfix, we should create the hotfix branch as soon as possible and merge all the fixes into it. I think it is the better work flow.

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.

5 participants