Skip to content

Fail things 2#28

Open
sergiou87 wants to merge 10 commits intodevelopmentfrom
fail-things-2
Open

Fail things 2#28
sergiou87 wants to merge 10 commits intodevelopmentfrom
fail-things-2

Conversation

@sergiou87
Copy link
Owner

N/A

friendlyName: Windows
timeout-minutes: 60
steps:
- name: Fail immediately # test 1 2 3 4 5 6
Copy link

Choose a reason for hiding this comment

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

Something

Copy link

Choose a reason for hiding this comment

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

Single comment

Copy link

Choose a reason for hiding this comment

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

test

Copy link

Choose a reason for hiding this comment

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

Multi-comment review

Copy link

Choose a reason for hiding this comment

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

This is me

Copy link

Choose a reason for hiding this comment

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

Hello Windows

Copy link

Choose a reason for hiding this comment

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

Hello?

Copy link

@serpa1 serpa1 left a comment

Choose a reason for hiding this comment

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

asdf

Copy link

@serpa1 serpa1 left a comment

Choose a reason for hiding this comment

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

Changes!!

Copy link

@serpa1 serpa1 left a comment

Choose a reason for hiding this comment

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

Approved

Copy link

@serpa1 serpa1 left a comment

Choose a reason for hiding this comment

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

asdfadsf

@serpa1
Copy link

serpa1 commented Jan 23, 2023

Test comment

Copy link

@serpa1 serpa1 left a comment

Choose a reason for hiding this comment

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

Global comment

@serpa1
Copy link

serpa1 commented Jan 23, 2023

Root comment

security-events: write

steps:
- name: Fail immediately # test
Copy link

Choose a reason for hiding this comment

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

Line comment

Copy link

Choose a reason for hiding this comment

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

Inline comment

Copy link

Choose a reason for hiding this comment

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

My comment

Copy link

Choose a reason for hiding this comment

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

Inline comment hello

Copy link

Choose a reason for hiding this comment

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

Another multicomment review

Copy link

Choose a reason for hiding this comment

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

inline comment somewhere

Copy link

Choose a reason for hiding this comment

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

inline comment somewhere (bis)

Copy link

@serpa1 serpa1 left a comment

Choose a reason for hiding this comment

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

Hello

@serpa1
Copy link

serpa1 commented Feb 14, 2023

This is a regular comment

@serpa1
Copy link

serpa1 commented Feb 14, 2023

Another comment

@serpa1
Copy link

serpa1 commented Feb 14, 2023

Moar comments?

@serpa1
Copy link

serpa1 commented Feb 14, 2023

Issue comment

@serpa1
Copy link

serpa1 commented Feb 14, 2023

Test comment

@serpa1
Copy link

serpa1 commented Feb 21, 2023

A comment

Copy link

@serpa1 serpa1 left a comment

Choose a reason for hiding this comment

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

Approved

Copy link

@serpa1 serpa1 left a comment

Choose a reason for hiding this comment

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

Changes plz

@serpa1
Copy link

serpa1 commented Feb 21, 2023

A comment?

@serpa1
Copy link

serpa1 commented Feb 21, 2023

Root comment for testing

Copy link

@serpa1 serpa1 left a comment

Choose a reason for hiding this comment

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

Review comment with body and no comments

Copy link

@serpa1 serpa1 left a comment

Choose a reason for hiding this comment

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

Approved!

@serpa1
Copy link

serpa1 commented Mar 9, 2023

Sup

@serpa1
Copy link

serpa1 commented Mar 9, 2023

Root comment for Windows 🥲

@serpa1
Copy link

serpa1 commented May 3, 2023

My comment

@@ -25,6 +25,9 @@ jobs:
friendlyName: Windows
timeout-minutes: 60
Copy link

Choose a reason for hiding this comment

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

New inline comment here

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.

3 participants