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

vault: update smoke test for recent code changes #2207

Conversation

ilandikov
Copy link
Collaborator

@ilandikov ilandikov commented Aug 14, 2023

Description

Update the smoke test instructions which were inaccurate after the PR #2047 was merged.

Motivation and Context

Have the same result as the expected in the smoke tests =)

How has this been tested?

Smoke tests! =)

Screenshots (if appropriate)

Снимок экрана 2023-08-14 в 10 15 25

Types of changes

Changes visible to users:

Checklist

Terms

@claremacrae claremacrae changed the title vault: fix priority headings in styling vault: update smoke test for recent code changes Aug 14, 2023
Copy link
Collaborator

@claremacrae claremacrae left a comment

Choose a reason for hiding this comment

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

Thanks. I noticed the styles mismatch recently and didn't make time to fix it.

@claremacrae claremacrae added type: internal Only regards development or contributing needs-smoke-test Automated tests are not enough: the smoke tests should be run on this please. labels Aug 14, 2023
@claremacrae claremacrae merged commit 107f94a into obsidian-tasks-group:main Aug 14, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-smoke-test Automated tests are not enough: the smoke tests should be run on this please. type: internal Only regards development or contributing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants