-
Notifications
You must be signed in to change notification settings - Fork 265
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
Scripts fixes to prevent teachers from continuously needing to input their PAT #369
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is fine
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reviewed all
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fine
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reviewed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed
. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed
merging on behalf @rajbos |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just noticed a small typo that can be addressed in a future PR.
@@ -269,8 +269,8 @@ create_issue() { | |||
|
|||
echo -n "Creating issue: $title... " | |||
|
|||
# Wait for 1 second to avoid secondary rate limiting | |||
sleep 1 | |||
# Wait for 2 second to avoid secondary rate limiting, 1 second was nog enough |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# Wait for 2 second to avoid secondary rate limiting, 1 second was nog enough | |
# Wait for 2 second to avoid secondary rate limiting (1 second was not enough) |
@@ -294,8 +294,8 @@ create_pull_request() { | |||
|
|||
echo -n "Creating pull request: $title... " | |||
|
|||
# Wait for 1 second to avoid secondary rate limiting | |||
sleep 1 | |||
# Wait for 2 seconds to avoid secondary rate limiting, 1 second was nog enough |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# Wait for 2 seconds to avoid secondary rate limiting, 1 second was nog enough | |
# Wait for 2 seconds to avoid secondary rate limiting (1 second was not enough) |
Tested with script 2, 4 and 6
There is also an extra test for the OSTYPE in here, so that the check works on Windows with WSL as well