Skip to content

Remove double "+"#721

Merged
ccarouge merged 1 commit intomainfrom
ccarouge-patch-1
Apr 17, 2026
Merged

Remove double "+"#721
ccarouge merged 1 commit intomainfrom
ccarouge-patch-1

Conversation

@ccarouge
Copy link
Copy Markdown
Member

@ccarouge ccarouge commented Apr 17, 2026

CABLE

Thank you for submitting a pull request to the CABLE Project.

Description

Remove 2 double "+" signs spotted during the sync of code with AM3.

Type of change

Please delete options that are not relevant.

  • Bug fix
  • New or updated documentation

Checklist

  • The new content is accessible and located in the appropriate section
  • I have checked that links are valid and point to the intended content
  • I have checked my code/text and corrected any misspellings

Testing

The code changes are all in the POP variables, so it can't be tested with benchcab. The CI shows the code compiles.

Please add a reviewer when ready for review.


📚 Documentation preview 📚: https://cable--721.org.readthedocs.build/en/721/

@ccarouge
Copy link
Copy Markdown
Member Author

@Whyborn I'm adding the same fix as what Ian spotted in AM3. Note, I found 2 occurrences with the extra "+".

@ccarouge ccarouge requested a review from Whyborn April 17, 2026 03:42
@ccarouge
Copy link
Copy Markdown
Member Author

If you have the time for a quick review @Whyborn since you know where this comes from.

Copy link
Copy Markdown
Contributor

@Whyborn Whyborn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hard to go wrong with this one

@ccarouge ccarouge merged commit 5148f8a into main Apr 17, 2026
5 checks passed
@ccarouge ccarouge deleted the ccarouge-patch-1 branch April 17, 2026 04:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants