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

fix sqale definitions #518

Merged
merged 1 commit into from
May 28, 2015
Merged

Conversation

jmecosta
Copy link
Member

jump version to 0.9.4 snashot improve some sqale difintions that cause some local analysis failure

@guwirth guwirth added the bug label May 24, 2015
@guwirth guwirth added this to the M 0.9.4 milestone May 24, 2015
@guwirth
Copy link
Collaborator

guwirth commented May 24, 2015

To merge it pom files (version 0.9.4) should be removed.

@jmecosta
Copy link
Member Author

Dont we increment the versions straight after a release?

On Sun, May 24, 2015, 17:07 Günter Wirth [email protected] wrote:

To merge it pom files (version 0.9.4) should be removed.


Reply to this email directly or view it on GitHub
#518 (comment).

@guwirth
Copy link
Collaborator

guwirth commented May 24, 2015

No, see https://github.com/wenns/sonar-cxx/wiki/Release-Procedure, that is the last step.

@jmecosta
Copy link
Member Author

We need to adjust this, usually we need to provide snapshot versions for
users to use to verify defects. We should not provide the same 0.9.3
because it has been released.

IMO the 0.9.4-snapashot should be the first commit after a release. The
same happens with all plugins in sonar community

On Sun, May 24, 2015, 17:52 Günter Wirth [email protected] wrote:

No, see https://github.com/wenns/sonar-cxx/wiki/Release-Procedure, that
is the last step.


Reply to this email directly or view it on GitHub
#518 (comment).

@guwirth
Copy link
Collaborator

guwirth commented May 24, 2015

Hi Jorge. Your are right I'm wrong. Didn't see the SNAPSHOT after the version number, sorry.

@jmecosta
Copy link
Member Author

But no need to merge already, let's see if someone can test this

On Sun, May 24, 2015, 17:59 Günter Wirth [email protected] wrote:

Hi Jorge. Your are right I'm wrong. Didn't see the SNAPSHOT after the
version number, sorry.


Reply to this email directly or view it on GitHub
#518 (comment).

@jmecosta jmecosta mentioned this pull request May 28, 2015
guwirth added a commit that referenced this pull request May 28, 2015
@guwirth guwirth merged commit 1e6e95b into SonarOpenCommunity:master May 28, 2015
@guwirth guwirth mentioned this pull request Oct 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants