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

IBX-8018: Fixed trailing whitespace in spellcheck #1224

Merged
merged 2 commits into from
Apr 10, 2024
Merged

IBX-8018: Fixed trailing whitespace in spellcheck #1224

merged 2 commits into from
Apr 10, 2024

Conversation

reithor
Copy link
Contributor

@reithor reithor commented Mar 27, 2024 •
edited
Loading

Question Answer
JIRA issue https://issues.ibexa.co/browse/IBX-8018
Type bug
Target version v4.6
BC breaks no
Doc needed no

Trim unneeded whitespace in spellcheck suggestion

Checklist:

  • Provided PR description.
  • Tested the solution manually.
  • Provided automated test coverage.
  • Checked that target branch is set correctly (master for features, the oldest supported for bugs).
  • Asked for a review (ping @ibexa/engineering).

@reithor reithor requested a review from adamwojs March 27, 2024 18:13
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@katarzynazawada katarzynazawada mentioned this pull request Apr 10, 2024
Closed
@mikadamczyk mikadamczyk merged commit b805ec0 into 4.6 Apr 10, 2024
22 checks passed
@mikadamczyk mikadamczyk deleted the ibx-8018-fixed-trailingwhitespace-in-spellcheck branch April 10, 2024 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@katarzynazawada katarzynazawada katarzynazawada approved these changes

@dew326 dew326 dew326 approved these changes

@mikadamczyk mikadamczyk mikadamczyk approved these changes

@adamwojs adamwojs adamwojs approved these changes

@Nattfarinn Nattfarinn Awaiting requested review from Nattfarinn

@ciastektk ciastektk Awaiting requested review from ciastektk

@ViniTou ViniTou Awaiting requested review from ViniTou

Assignees

@mikadamczyk mikadamczyk

@Nattfarinn Nattfarinn

@ciastektk ciastektk

@dew326 dew326

@ViniTou ViniTou

Labels
QA approved Ready for review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@reithor @adamwojs @mikadamczyk @dew326 @katarzynazawada @Nattfarinn @ciastektk @ViniTou