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

add SQ 5.3-RC1 to Travis CI testing #707

Merged
merged 1 commit into from
Dec 12, 2015
Merged

add SQ 5.3-RC1 to Travis CI testing #707

merged 1 commit into from
Dec 12, 2015

Conversation

guwirth
Copy link
Collaborator

@guwirth guwirth commented Dec 12, 2015

No description provided.

@guwirth guwirth self-assigned this Dec 12, 2015
@guwirth guwirth added this to the M 0.9.5 milestone Dec 12, 2015
@jmecosta
Copy link
Member

5.2 will never support the current plugin, so we can remove that. if they release 5.2.1 then it can be added.

for 5.3 the current its will require a modification to wait for server analysis to end before checking the analysis, but otherwise it should be ok

@jmecosta
Copy link
Member

and it seems it requires bigger refactoring to support the new rest api.

    File "integration-tests/features/steps/test_execution_statistics.py", line 206, in assert_measures
      json_measures = json.loads(response.text)[0].get("msr", None)
  KeyError: 0

@guwirth
Copy link
Collaborator Author

guwirth commented Dec 12, 2015

@jmecosta merge this anyway to see the current state.

and it seems it requires bigger refactoring to support the new rest api.

Have less knowledge here. Can you do this?

guwirth added a commit that referenced this pull request Dec 12, 2015
add SQ 5.3-RC1 to Travis CI testing
@guwirth guwirth merged commit d646ffb into SonarOpenCommunity:master Dec 12, 2015
@jmecosta
Copy link
Member

Yep, its OK to merge. I will take a look during next week

On Sun, Dec 13, 2015, 00:45 Günter Wirth [email protected] wrote:

Merged #707 #707.


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

@guwirth guwirth deleted the enhancement/sq-5.3 branch April 3, 2016 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants