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

Update workflow syntax #60626

Merged
merged 22 commits into from
Mar 19, 2020
Merged

Update workflow syntax #60626

merged 22 commits into from
Mar 19, 2020

Conversation

alexh97
Copy link

@alexh97 alexh97 commented Mar 19, 2020

Summary

Updating workflow syntax to work with the new action version. Forgot to do this in #60577

Alex Holmansky and others added 21 commits January 31, 2020 12:36
Use diagnostics-enable action in the workflow. Issue: elastic#56526
130d4e5
Merge remote-tracking branch 'upstream/master'
b616992
Merge remote-tracking branch 'upstream/master'
1e47009
Update workflow to use v1.0.2 of the action
fcca5a9
Adding a new test workflow that uses a personal access token
b8dde41
Syncing the fork.
656cedc
Remove an extra coma
751d84c
@elasticmachine
Merge branch 'master' into master
2f10f03
@elasticmachine
Merge branch 'master' into master
4eba121
Syncing the fork.
5a67d3a
Updated project-assigner action version and access key
cc5cb53
Deleted the test workflow
caeca48
Syncing the fork.
998bddd
Syncing the fork
7b2211f
Temporarily commenting out project mappings while we debug the permis…
99c005e 
…sions issues
Sync the fork.
476d8ae
Temporarily commenting out project mappings while we debug the permis…
649385e 
…sions issues
Syncing the fork.
03d6e09
Updating workflows to v2.0.0 of the action and back to the default token
e341dca
Making a small change to the readme to create a test PR
aa65f31
Updated workflow syntax
471e98b
@alexh97 alexh97 added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes labels Mar 19, 2020
@alexh97 alexh97 requested a review from tylersmalley March 19, 2020 13:56
@alexh97 alexh97 requested a review from a team as a code owner March 19, 2020 13:56
tylersmalley
tylersmalley approved these changes Mar 19, 2020
View reviewed changes
Copy link
Contributor

@tylersmalley tylersmalley left a comment

Choose a reason for hiding this comment

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

ID > Names - LGTM!

@alexh97 alexh97 merged commit 80a8407 into elastic:master Mar 19, 2020
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Mar 20, 2020
@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create backports run node scripts/backport --pr 60626 or prevent reminders by adding the backport:skip label.

gmmorris added a commit to gmmorris/kibana that referenced this pull request Mar 20, 2020
@gmmorris
Merge branch 'master' into alerting/tls-warning
becff42 
* master:
  [ML] Use a new ML endpoint to estimate a model memory (elastic#60376)
  [Logs UI] Correctly update the expanded log rate table rows (elastic#60306)
  fixes drag and drop flakiness (elastic#60625)
  Removing isEmptyState from embeddable input (elastic#60511)
  [Cross Cluster Replication] NP Shim (elastic#60121)
  Clear changes when canceling an edit to an alert (elastic#60518)
  Update workflow syntax (elastic#60626)
  Updating project assigner workflows to v2.0.0 of the action and back to default tokens (elastic#60577)
  migrate saved objects management edition view to react/typescript/eui (elastic#59490)
@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create backports run node scripts/backport --pr 60626 or prevent reminders by adding the backport:skip label.

4 similar comments
@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create backports run node scripts/backport --pr 60626 or prevent reminders by adding the backport:skip label.

@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create backports run node scripts/backport --pr 60626 or prevent reminders by adding the backport:skip label.

@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create backports run node scripts/backport --pr 60626 or prevent reminders by adding the backport:skip label.

@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create backports run node scripts/backport --pr 60626 or prevent reminders by adding the backport:skip label.

@tylersmalley tylersmalley added the backport:skip This commit does not require backporting label Mar 26, 2020
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Mar 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tylersmalley tylersmalley tylersmalley approved these changes

Assignees
No one assigned
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes v8.0.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@alexh97 @kibanamachine @tylersmalley @elasticmachine