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.

Sign up for GitHub

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

Jump to bottom

Fix secondary location-based rules: encode secondary locations conditionally #304

Merged
merged 4 commits into from
Nov 22, 2021
Merged

Fix secondary location-based rules: encode secondary locations conditionally #304

merged 4 commits into from
Nov 22, 2021

Conversation

yassin-kammoun-sonarsource
Copy link
Contributor

Fixes #287

@yassin-kammoun-sonarsource yassin-kammoun-sonarsource changed the title Fix no-gratuitous-expressions, no-same-line-conditional: encode secondary locations conditionally Fix secondary location-based rules: encode secondary locations conditionally Nov 22, 2021
@sonarcloud
Copy link

sonarcloud bot commented Nov 22, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@saberduck saberduck merged commit c0508f2 into master Nov 22, 2021
@saberduck saberduck deleted the issue-287 branch November 22, 2021 16:11
@yassin-kammoun-sonarsource yassin-kammoun-sonarsource mentioned this pull request Nov 22, 2021
Closed
This was referenced Dec 1, 2021
Merged
Merged
Merged
@deepcrawltech deepcrawltech mentioned this pull request Dec 1, 2021
Closed
1 task
@renovate renovate bot mentioned this pull request Dec 1, 2021
Closed
1 task
@deepcrawltech deepcrawltech mentioned this pull request Dec 1, 2021
Open
1 task
This was referenced Dec 1, 2021
Merged
Merged
Closed
Merged
Merged
Merged
Closed
Merged
Merged
Merged
Merged
Merged
Merged
This was referenced Dec 2, 2021
Merged
Merged
Merged
Merged
Merged
Merged
Merged
@renovate renovate bot mentioned this pull request Dec 10, 2021
Merged
1 task
This was referenced Dec 17, 2021
Merged
Merged
Closed
Closed
This was referenced Dec 31, 2021
Merged
Merged
This was referenced Jan 29, 2022
Merged
Merged
This was referenced Mar 7, 2022
Open
Open
Open
Merged
Open
Merged
This was referenced Mar 31, 2022
Merged
Merged
This was referenced Apr 11, 2022
Merged
Merged
Open
@renovate renovate bot mentioned this pull request Apr 22, 2022
Closed
1 task
This was referenced May 1, 2022
Closed
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@saberduck saberduck saberduck approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Raw message is shown as JSON instead of showing the message text
3 participants
@yassin-kammoun-sonarsource @saberduck @a-tarasyuk