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

change to test some functionality #74

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

Conversation

etpalmer63
Copy link
Collaborator

Try typing "Do.Tests" to initiate tests on this repo.

@WeiqunZhang
Copy link
Member

    Do.Tests  

@WeiqunZhang
Copy link
Member

I intentionally inserted a number of spaces before and after Do.Tests. Would this comment trigger testing too?

@etpalmer63
Copy link
Collaborator Author

Looks like that one didn't work

@WeiqunZhang
Copy link
Member

Do.Tests How after space after it?

@etpalmer63
Copy link
Collaborator Author

I think the comment needs to be "Do.Tests" and nothing else, but let me double check

@WeiqunZhang
Copy link
Member

What about new lines by Enter?

@WeiqunZhang
Copy link
Member

So what exactly is the match syntax?

@WeiqunZhang
Copy link
Member

Do.Tests

@etpalmer63
Copy link
Collaborator Author

The comment needs to be exactly "Do.Tests" and nothing else --> github.event.comment.body == env.TRIGGER_STRING

@WeiqunZhang
Copy link
Member

Do.Tests

@github-actions
Copy link

Initiated Cori Compilation

@WeiqunZhang
Copy link
Member

Seems to work

@etpalmer63
Copy link
Collaborator Author

From here it's taking about 20 mins to complete the CI.

@etpalmer63
Copy link
Collaborator Author

I have concerns about how it could break .. I use the PR comment to find the branch, so if there are a lot of comments at once, I'm worried it might grab the wrong one. I'm also concerned about triggering it multiple times before it finishes the previous task.

I have been looking for some example bots to see how they handle these things but haven't found a good example yet.

@etpalmer63
Copy link
Collaborator Author

HeatEquation -- Cori Compile Success

@etpalmer63
Copy link
Collaborator Author

CNS_SOD -- Cori Compile Success

@etpalmer63
Copy link
Collaborator Author

CNS_RT -- Cori Compile Success

@asalmgren
Copy link
Member

I'd also like to comment on this PR ...

@asalmgren
Copy link
Member

Do.Tests

1 similar comment
@etpalmer63
Copy link
Collaborator Author

Do.Tests

@github-actions
Copy link

Initiated Cori Compilation

@etpalmer63
Copy link
Collaborator Author

HeatEquation -- Cori Compile Success

@etpalmer63
Copy link
Collaborator Author

CNS_SOD -- Cori Compile Success

@etpalmer63
Copy link
Collaborator Author

CNS_RT -- Cori Compile Success

@etpalmer63
Copy link
Collaborator Author

Test

1 similar comment
@etpalmer63
Copy link
Collaborator Author

Test

@atmyers
Copy link
Member

atmyers commented Jun 27, 2022

Do.Tests

@github-actions
Copy link

Tests Initiated.

@etpalmer63
Copy link
Collaborator Author

HeatEquation -- Cori Compile Success

@etpalmer63
Copy link
Collaborator Author

Do.Tests

@github-actions
Copy link

Tests Initiated.

@etpalmer63
Copy link
Collaborator Author

CNS_SOD -- Cori Compile Success

@etpalmer63
Copy link
Collaborator Author

HeatEquation -- Cori Compile Success

@etpalmer63
Copy link
Collaborator Author

CNS_RT -- Cori Compile Success

@etpalmer63
Copy link
Collaborator Author

CNS_SOD -- Cori Compile Success

@etpalmer63
Copy link
Collaborator Author

CNS_RT -- Cori Compile Success

@jmsexton03
Copy link
Collaborator

Do.Tests

@github-actions
Copy link

Tests Initiated.

@etpalmer63
Copy link
Collaborator Author

HeatEquation -- Cori Compile Success

@etpalmer63
Copy link
Collaborator Author

CNS_SOD -- Cori Compile Success

@etpalmer63
Copy link
Collaborator Author

CNS_RT -- Cori Compile Success

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.

5 participants