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

Release build script #1231

Merged
merged 6 commits into from
Sep 23, 2018
Merged

Release build script #1231

merged 6 commits into from
Sep 23, 2018

Conversation

jim-parry
Copy link
Contributor

And .... here we go. This is the first crack at the release builder.
It is tested to the "are we there yet?" point, but I am reluctant to proceed with the next level of testing until someone else has taken a peek, to make sure I didn't do somthing stupid.

@jim-parry jim-parry added this to the 4.0.0-alpha milestone Sep 22, 2018
admin/release Show resolved Hide resolved
admin/release Show resolved Hide resolved
@jim-parry
Copy link
Contributor Author

I plan to merge this, to get the ball rolling, but to only use the release branch building part for the first release - doing the deployment manually to make sure I didn't forget or mess up anything. The script can be adjusted once we see how that works.

One thing unclear - whether the script will make a pre-release or a full release, from github's perspective. Do I need to do something like the solution here, with a curl post inside the script?

@jim-parry jim-parry merged commit 5a66516 into codeigniter4:develop Sep 23, 2018
@jim-parry jim-parry deleted the admin/build branch September 23, 2018 15:21
@lonnieezell
Copy link
Member

@jim-parry that sounds like a good idea. Nice work, btw.

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.

None yet

3 participants