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

fix release action #212

Merged
merged 5 commits into from
Sep 24, 2021
Merged

fix release action #212

merged 5 commits into from
Sep 24, 2021

Conversation

mofojed
Copy link
Member

@mofojed mofojed commented Sep 24, 2021 •
edited
Loading

  • Remove unnecessary uses flag
  • Pass the GITHUB_AUTH as an env var in the run command

@mofojed mofojed requested a review from devinrsmith September 24, 2021 19:15
@mofojed mofojed self-assigned this Sep 24, 2021
@mofojed mofojed added this to the September 2021 milestone Sep 24, 2021
@mofojed mofojed merged commit 84aa153 into deephaven:main Sep 24, 2021
@mofojed mofojed deleted the fix-release-action branch September 24, 2021 19:31
@mofojed mofojed mentioned this pull request Sep 24, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@devinrsmith devinrsmith devinrsmith approved these changes

Assignees

@mofojed mofojed

Labels
internal web-client-ui
Projects
None yet
Milestone
September 2021
Development

Successfully merging this pull request may close these issues.
2 participants
@mofojed @devinrsmith