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.

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

fix: disable comments on Jest testing #51

Merged
merged 10 commits into from
Jun 24, 2024

Conversation

gentlementlegen
Copy link
Member

@gentlementlegen gentlementlegen commented Jun 21, 2024

Resolves #50

@gentlementlegen gentlementlegen changed the base branch from main to development June 21, 2024 16:58
@ubiquity-os-deployer
Copy link

ubiquity-os-deployer bot commented Jun 21, 2024

@gentlementlegen gentlementlegen marked this pull request as draft June 21, 2024 17:01
@gentlementlegen
Copy link
Member Author

gentlementlegen commented Jun 21, 2024

No more comment is posted, only the result can be visible within the Action run itself: https://github.com/ubiquity/ts-template/actions/runs/9617643273/job/26529830238?pr=51#step:4:41

If that is satisfactory, I will also apply this within https://github.com/ubiquibot/plugin-template


I do not know if that is interesting to have but we could also add the summary on the Action run like so: https://github.com/Meniole/ts-template/actions/runs/9617961749/attempts/1#summary-26530846974

@gentlementlegen gentlementlegen changed the title merge main fix: disable comments on Jest testing Jun 21, 2024
@0x4007
Copy link
Member

0x4007 commented Jun 21, 2024

Summary on the action seems like a good solution. Just as long as you verify that it doesn't produce a notification.

@gentlementlegen gentlementlegen marked this pull request as ready for review June 22, 2024 00:04
@gentlementlegen
Copy link
Member Author

gentlementlegen commented Jun 22, 2024

@0x4007 I should hopefully not. If it ever does, I will remove this section right away. But afaik, since it's just tied to the Action run itself, it should not.

This run has the summary linked to itself.

@0x4007 0x4007 merged commit 9fdf38e into ubiquity:development Jun 24, 2024
3 checks passed
@@ -7,5 +7,6 @@ describe("Homepage tests", () => {
});
cy.get("@consoleError").should("not.be.called");
cy.get("body").should("exist");
cy.get("h1").should("exist");
Copy link
Member

Choose a reason for hiding this comment

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

Why was this added?

Copy link
Member Author

Choose a reason for hiding this comment

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

Not very relevant to this PR, just adding more coverage while I was testing, so I thought it could remain there.

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.

Disable Coverage Report Comment
2 participants