Skip to content

Implemented new workflows#4

Open
m-messer wants to merge 6 commits intomainfrom
feature/workflows
Open

Implemented new workflows#4
m-messer wants to merge 6 commits intomainfrom
feature/workflows

Conversation

@m-messer
Copy link
Copy Markdown
Member

@m-messer m-messer commented May 6, 2026

Implement the new workflow process that has been developed in PR 12 of the workflows repo.

…duction deployment; removed old test report. Updated README with deployment instructions.
Copy link
Copy Markdown
Collaborator

@neagualexa neagualexa left a comment

Choose a reason for hiding this comment

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

I don't fully understand when the pre_production_tests run (update: I've just reviewed the PR on the workflows repo and I found it being used there, but when is it used? as run-database-tests is false ).
The rest looks good.


try:
return Result(preview=Preview(sympy=response))
except FeedbackException as e:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

is this change supposed to be in this PR?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes, mainly as it was causing issues with the CI.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

For the pre-prod tests it can be ran through the UI on GitHub.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

i see that is clear thanks

@neagualexa neagualexa self-requested a review May 6, 2026 13:06
@neagualexa neagualexa assigned m-messer and unassigned neagualexa May 6, 2026
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