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.

Sign up for GitHub

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

Jump to bottom

Multiplication was used incorrectly in examles #3278

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Multiplication was used incorrectly in examles #3278

wants to merge 1 commit into from

Conversation

lukaszachy
Copy link
Collaborator

Leading to

Invalid yaml syntax: Failed to parse 'test.fmf'.
found undefined alias
 in "<unicode string>", line 6, column 13

Pull Request Checklist

  • write the documentation

@lukaszachy
Multiplication was used incorrectly in examles
22cd965 
Leading to
```
Invalid yaml syntax: Failed to parse 'test.fmf'.
found undefined alias
 in "<unicode string>", line 6, column 13
```
@lukaszachy lukaszachy added documentation Improvements or additions to documentation code | no functional change "No Functional Change" intended. Patch should not change tmt's behavior in any way. labels Oct 10, 2024
@lukaszachy lukaszachy added this to the 1.38 milestone Oct 10, 2024
@pellecchialuigi
Copy link

Thanks a lot!

@lukaszachy lukaszachy linked an issue Oct 10, 2024 that may be closed by this pull request
Update tmt docs to show correct syntax for duration multiplication. Current syntax gives parsing error. #3272
Open
happz
happz reviewed Oct 10, 2024
View reviewed changes
spec/tests/duration.fmf
@@ -12,8 +12,12 @@ description: |
First, all time values are summed together, and only then are they multiplied.
The final value is then rounded up to the whole number.

Note that the asteric has a special meaning in YAML syntax and thus you need to
Copy link
Collaborator

Choose a reason for hiding this comment

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

Typo, it's "asterisk". I'd also add "... asterisc character, "*", ..."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@happz happz happz left review comments

@psss psss Awaiting requested review from psss psss is a code owner

@thrix thrix Awaiting requested review from thrix thrix is a code owner

@janhavlin janhavlin Awaiting requested review from janhavlin janhavlin is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
code | no functional change "No Functional Change" intended. Patch should not change tmt's behavior in any way. documentation Improvements or additions to documentation
Projects
None yet
Milestone
1.38
Development

Successfully merging this pull request may close these issues.

Update tmt docs to show correct syntax for duration multiplication. Current syntax gives parsing error.
3 participants
@lukaszachy @pellecchialuigi @happz