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

Publish only tagged packages #263

Merged
merged 2 commits into from Mar 13, 2017

Conversation

marcjansen
Copy link
Member

@marcjansen marcjansen commented Mar 13, 2017

This PR is WIP until we have released v3.0.0.

Previously we would publish a sencha package for every commit that
happened in the master-branch. This commit changes this in a way
that only tagged commits (released versions) will publish a sencha
package.

We still need to define how we actually want to react when we build
for say v3.0.1. The current approach would probably make the
originally published v3.0.0 unusable (because we rm probably too
much, and also copy over catalog.json metatdata files where we
actually want to merge new catalog.json with existing ones).

Comit 21299d8 temporary disables the publishing alltogether,
so we cannot break any existing published package.

Previously we would publish a sencha package for every commit that
happened in the `master`-branch. This commit changes this in a way
that only tagged commits (released versions) will publish a sencha
package.

We still need to define how we actually want to react when we build
for say `v3.0.1`. The current approach would probably make the
originally published `v3.0.0` unusable (because we `rm` probably too
much, and also copy over `catalog.json` metatdata files where we
actually want to merge new `catalog.json` with existing ones).

An upcoming commit will temporary disable the publishing alltogether,
so we cannot break the existing published package.
We disable the publishing of the sencha package until we know
what we need to do in order to be able to publish e.g. `v3.0.1`
when we already have a `v3.0.0`.
@marcjansen marcjansen changed the title [WIP] Publish only tagged packages Publish only tagged packages Mar 13, 2017
@marcjansen
Copy link
Member Author

We have v3.0.0 now, I'll merge this now as a safeguard.

@marcjansen marcjansen merged commit c743992 into geoext:master Mar 13, 2017
@marcjansen marcjansen deleted the publish-only-tagged-packages branch March 13, 2017 19:40
@marcjansen marcjansen mentioned this pull request Mar 13, 2017
3 tasks
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

1 participant