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

GitHub actions #190

Merged
merged 24 commits into from
May 17, 2022
Merged

Conversation

AlexNDRmac
Copy link
Contributor

Hi @NoiseByNorthwest. This is first part of #180

This is base workflow which includes cross-php version builds for mac/linux OS.

Features

  • Now each build will store build artifact to workflow summary. It's very useful, users don't need to compile extension - just download compiled and use!
  • If smth goes wrong - CI workflow have an action step to store logs for the failed step which helps to sort out with possible mistakes.
  • CI have step to check the release notes parser - this is useful to check if everything is OK with release notes before creating new Release.
  • Created CHANGELOG with all the history of changes.
  • Created release notes parser from changelog to simplify Release notes creation. Now, all you need - just push the tag with for new version or simply create new release with empty release notes body - Release action will do everything else. Release action will run CI to ensure that everything is OK, then download all the build artifacts, parse release notes for the latest TAG in CHANGELOG.md and creates Release with this content and all the build artifacts (extensions for all PHP version and all supported OS)

What i missed

  • I'm not removed Travis workflow file - please remove it yourself after github actions successful merge
  • I'm do not add new version to CHANGELOG, only add changes to Unreleased section. You can easily create new version yourself when you decide - just move everything from Unreleased to new version section and change link to compare changes in H2 markdown tag

Thanks

env:
ZIP_EXCLUDE: -x ".*" -x "__MACOSX" -x "*.DS_Store"

- name: Check Release notes
Copy link
Owner

Choose a reason for hiding this comment

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

What is the point of this step in this context ?

Copy link
Owner

Choose a reason for hiding this comment

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

OK, to check that the script do not fail before entering the release process.

@NoiseByNorthwest
Copy link
Owner

@AlexNDRmac Thanks for your work, LGTM.

Do I have to merge first in order to enable github actions ?

@AlexNDRmac
Copy link
Contributor Author

@NoiseByNorthwest - yes, to enable actions, you should merge first. After merge, actions will start and status badge also should retrieve status automatically.

@NoiseByNorthwest NoiseByNorthwest merged commit 139535e into NoiseByNorthwest:master May 17, 2022
@AlexNDRmac AlexNDRmac deleted the github-actions branch September 13, 2022 09:12
@AlexNDRmac AlexNDRmac mentioned this pull request Sep 28, 2022
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.

2 participants