Skip to content

Conversation

@leplatrem
Copy link
Contributor

@leplatrem leplatrem commented Feb 21, 2024

@leplatrem leplatrem force-pushed the leverage-dockerflow-2024.2.0 branch from 68d4f3d to 621c816 Compare February 22, 2024 17:22
@leplatrem
Copy link
Contributor Author

@grahamalama do you want to cherry-pick and add your commit 643bee9 on top of this?

@leplatrem leplatrem changed the title Leverage dockerflow 2024.2.0 Leverage dockerflow 2024.3.0 Mar 6, 2024
@leplatrem leplatrem force-pushed the leverage-dockerflow-2024.2.0 branch from 8aa07c5 to df4d3ce Compare March 6, 2024 12:31
@leplatrem leplatrem marked this pull request as ready for review March 6, 2024 12:32
@leplatrem leplatrem requested a review from a team as a code owner March 6, 2024 12:32
@leplatrem leplatrem force-pushed the leverage-dockerflow-2024.2.0 branch from df4d3ce to 191b5d8 Compare March 6, 2024 12:33
return []

def check_jira_pandoc_install(self):
if markdown_to_jira("- Test") != "* Test":
Copy link
Contributor

Choose a reason for hiding this comment

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

I would have assumed we'd get an exception rather than an invalid string being returned.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hmm, you made me doubt... if an exception is raised then the endpoint returns 500 as expected

Copy link
Contributor

@alexcottner alexcottner left a comment

Choose a reason for hiding this comment

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

Looks like we're doing better things with less code

@leplatrem leplatrem enabled auto-merge (squash) March 7, 2024 10:38
@leplatrem leplatrem merged commit 3d13206 into main Mar 7, 2024
@leplatrem leplatrem deleted the leverage-dockerflow-2024.2.0 branch March 7, 2024 10:40
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.

4 participants