-
Notifications
You must be signed in to change notification settings - Fork 228
Add slash command '/test-gmt-dev' to test GMT dev version #831
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
Changes from 5 commits
11446e2
4a22c59
8620bab
ef31c15
3c810f5
6df8e34
72b7dfa
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -7,7 +7,9 @@ on: | |
# push: | ||
# branches: [ master ] | ||
pull_request: | ||
types: [review_requested, ready_for_review] | ||
types: [ready_for_review] | ||
repository_dispatch: | ||
types: [test-gmt-master-command] | ||
seisman marked this conversation as resolved.
Show resolved
Hide resolved
|
||
# Schedule daily tests | ||
schedule: | ||
- cron: '0 0 * * *' | ||
|
@@ -36,10 +38,31 @@ jobs: | |
# Checkout current git repository | ||
- name: Checkout | ||
uses: actions/[email protected] | ||
if: github.event_name != 'repository_dispatch' | ||
with: | ||
# fecth all history so that setuptools-scm works | ||
fetch-depth: 0 | ||
|
||
# Generate token from GenericMappingTools bot | ||
- name: Generate token from GenericMappingTools bot | ||
uses: tibdex/github-app-token@v1 | ||
if: github.event_name == 'repository_dispatch' | ||
id: generate-token | ||
with: | ||
app_id: ${{ secrets.APP_ID }} | ||
private_key: ${{ secrets.APP_PRIVATE_KEY }} | ||
|
||
# Checkout the pull request branch | ||
- name: Checkout | ||
uses: actions/checkout@v2 | ||
if: github.event_name == 'repository_dispatch' | ||
with: | ||
token: ${{ steps.generate-token.outputs.token }} | ||
repository: ${{ github.event.client_payload.pull_request.head.repo.full_name }} | ||
ref: ${{ github.event.client_payload.pull_request.head.ref }} | ||
# fecth all history so that setuptools-scm works | ||
fetch-depth: 0 | ||
|
||
# Setup Miniconda | ||
- name: Setup Miniconda | ||
uses: conda-incubator/[email protected] | ||
|
@@ -106,3 +129,12 @@ jobs: | |
with: | ||
name: artifact-GMT-${{ matrix.gmt_git_ref }}-${{ runner.os }} | ||
path: tmp-test-dir-with-unique-name | ||
|
||
- name: Add reaction | ||
uses: peter-evans/create-or-update-comment@v1 | ||
if: github.event_name == 'repository_dispatch' | ||
with: | ||
token: ${{ steps.generate-token.outputs.token }} | ||
repository: ${{ github.event.client_payload.github.payload.repository.full_name }} | ||
comment-id: ${{ github.event.client_payload.github.payload.comment.id }} | ||
reaction-type: hooray |
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -21,5 +21,6 @@ jobs: | |||||
token: ${{ steps.generate-token.outputs.token }} | ||||||
commands: | | ||||||
format | ||||||
test-gmt-master | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
What do you think about changing the name to 'latest' to match the Github Actions name? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. When we say "the latest GMT", we usually mean "the latest stable GMT release" (i.e., 6.1.1). What about "GMT Dev Tests" and There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yep, dev sounds good and is shorter. Please remember to change the PR title too. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||||||
issue-type: pull-request | ||||||
permission: none |
Uh oh!
There was an error while loading. Please reload this page.