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

v1.0 Integration work #200

Merged
merged 202 commits into from
Dec 5, 2021

Conversation

andybak and others added 30 commits April 14, 2021 15:28
…d add more turtles. I like turtles.

[CI BUILD]
In order to make it easier to run multiple versions, instead of a fixed
"-github" name (and package name of com.icosa.openbrushgithub), this
commit changes the logic for CI builds.

* Builds from PRs are named PR#xx (PRxx in the package name)
* Builds from branches (triggered with a [CI BUILD] tag) are named based on the branch
name. Special characters of /[#_-]/ are removed from the package name
* All commits on main have the same name, whether they're pre-release
builds or formal release builds. This means that the pre-release builds
will NOT have a unique name, but this will allow us to upload them to a
"beta" release channel
[CI BUILD]
[CI BUILD]
[CI BUILD]
[CI BUILD]
…ands to endpoints where appropriate.

[CI BUILD]
Assets/Editor/BrushTagger.cs Outdated Show resolved Hide resolved
@mikeage mikeage changed the title Features/moatmerge plus pr198 v1.0 Integration work Dec 5, 2021
Copy link
Member

@mikeage mikeage left a comment

Choose a reason for hiding this comment

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

Seems to work for me. So all I can do is link to https://twitter.com/iamdevloper/status/397664295875805184

@mikeskydev mikeskydev merged commit 56a5337 into icosa-foundation:main Dec 5, 2021
@andybak andybak deleted the features/moatmerge-plusPR198 branch December 3, 2022 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Feature added
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants