Skip to content

Add Misc updates#3662

Merged
AyanSinhaMahapatra merged 6 commits intodevelopfrom
misc-updates
Mar 15, 2024
Merged

Add Misc updates#3662
AyanSinhaMahapatra merged 6 commits intodevelopfrom
misc-updates

Conversation

@pombredanne
Copy link
Member

@pombredanne pombredanne commented Feb 21, 2024

This PR contains miscellaneous small changes and fixes in tests, regex syntax and similar.
Thee are too small to be considered alone.

Tasks

  • Reviewed contribution guidelines
  • PR is descriptively titled 📑 and links the original issue above 🔗
  • Tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR
    Run tests locally to check for errors.
  • Commits are in uniquely-named feature branch and has no merge conflicts 📁
  • Updated documentation pages (if applicable)
  • Updated CHANGELOG.rst (if applicable)

Signed-off-by: Philippe Ombredanne <[email protected]>
Signed-off-by: Philippe Ombredanne <[email protected]>
Also use proper name for var: test_envt confuse test discovery

Signed-off-by: Philippe Ombredanne <[email protected]>
Signed-off-by: Philippe Ombredanne <[email protected]>
Some tests never ran and the underlying needed some minor updates.

Signed-off-by: Philippe Ombredanne <[email protected]>
Reference: #3667
Reported-by: Stefan @stefan6419846
Signed-off-by: Philippe Ombredanne <[email protected]>
Copy link
Member Author

@pombredanne pombredanne left a comment

Choose a reason for hiding this comment

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

All green. @AyanSinhaMahapatra yours to review and merge

Copy link
Member

@AyanSinhaMahapatra AyanSinhaMahapatra left a comment

Choose a reason for hiding this comment

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

Thanks @pombredanne
Merging this!

@AyanSinhaMahapatra AyanSinhaMahapatra merged commit ea8953b into develop Mar 15, 2024
@pombredanne pombredanne deleted the misc-updates branch May 9, 2024 07:08
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