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

Fix grammar in message when switching workspaces #58

Merged
merged 1 commit into from
Jul 11, 2020

Conversation

mbrukman
Copy link
Contributor

  • replace ! with ?
  • add missing space between sentences
  • remove extra space between words

* replace `!` with `?`
* add missing space between sentences
* remove extra space between words
@mbrukman
Copy link
Contributor Author

@rrmerugu — looks like several actions failed, but I think only you can fix this as the administrator.

https://github.com/invanalabs/graph-explorer/pull/58/checks?check_run_id=859752211 shows:

fatal: 'heroku' does not appear to be a git repository
fatal: Could not read from remote repository.

https://github.com/invanalabs/graph-explorer/actions/runs/165115922 shows:

Resource not accessible by integration

which suggests that this action doesn't have permissions to post a comment on the PR.

Can you take a look at these errors, please?

@rrmerugu
Copy link
Member

Appreciate the PRs @mbrukman .
Apparently there is a bug with "first interactions" Github action. This happens when the person creating the PR does not have permission to the repository actions/first-interaction#10 apache/airflow#7035. I think Deploy to Heroku must be similar issue.

Apparently the permission requirements for Github actions are not scalable for this usecase. I will look for alternatives to do the automation chores.

@rrmerugu
Copy link
Member

rrmerugu commented Jul 11, 2020

Meanwhile I will disable the Github actions and merge this PR

@rrmerugu rrmerugu merged commit 93e8511 into invana:master Jul 11, 2020
@mbrukman mbrukman deleted the patch-1 branch July 11, 2020 18:47
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.

2 participants