Skip to content
This repository was archived by the owner on Jan 7, 2021. It is now read-only.

Conversation

@maksat-software
Copy link

No description provided.

@t-8ch
Copy link
Contributor

t-8ch commented Feb 18, 2019

Hi @maksat-software ,

thanks for this!
Currently this is introducing a lot of whitespace changes. Could you minimize these?

@t-8ch
Copy link
Contributor

t-8ch commented Feb 18, 2019

Also it seems to me, this PR breaks compat with SQ 6.7.
sonar-stash tries to preserve compatibility with the latest LTS version of SQ, which currently is 6.7.
Do you think it would be possible to support both 6.7 and 7.6 at the same time?

@t-8ch
Copy link
Contributor

t-8ch commented Feb 18, 2019

Also, the release file should not be part of the source repository.

@t-8ch t-8ch mentioned this pull request Feb 19, 2019
@t-8ch
Copy link
Contributor

t-8ch commented Feb 19, 2019

@maksat-software I created a new PR #199 that

  • Implements support for 7.6, without dropping 6.7
  • Adds a new integrationtest to make validate multi-module projects

If you want, you can test it.
I will update the dependencies separately.

@ghost ghost closed this by deleting the head repository Oct 20, 2025
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants