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 contributing shellcheck #240

Merged
merged 3 commits into from
Sep 9, 2021
Merged

Conversation

Bill-Kunj
Copy link
Collaborator

No description provided.

Copy link
Contributor

@dckc dckc left a comment

Choose a reason for hiding this comment

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

I'm not interested to maintain shell scripts; I'd rather get rid of them rather than document them more. I'll leave this for someone else to reivew.

@Bill-Kunj
Copy link
Collaborator Author

Bill-Kunj commented Jul 13, 2021

I'm not interested to maintain shell scripts; I'd rather get rid of them rather than document them more. I'll leave this for someone else to reivew.

@dckc Getting rid of them is the reason for #217. Help is needed to figure that out. #217 simplifies the re-implementation of the scripts, and the implementation of a non-developer interface for RGOV.
A failed attempt was already made to refactor and re-implement in JavaScript in #116 and #143. There seemed to be no interest.

@Bill-Kunj Bill-Kunj merged commit 3ed4125 into master Sep 9, 2021
@Bill-Kunj Bill-Kunj deleted the fix-contributing-shellcheck branch September 9, 2021 04:11
Copy link
Contributor

@dckc dckc left a comment

Choose a reason for hiding this comment

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

In what way is this a fix? What was broken?

@@ -8,6 +8,8 @@ This project uses [```shellcheck```](https://github.com/koalaman/shellcheck) for
Shell script contributors using vscode are encouraged to install the vscode extension https://marketplace.visualstudio.com/items?itemName=timonwong.shellcheck.
```shellcheck``` is also available in the Ubuntu package ```shellcheck```

[We](https://docs.google.com/document/d/1UfoBMB_OFm-pkjid0u9d0GddB2bWXjc8u-bo4Es0isU/edit#) also suggest all contributions should pass:
Copy link
Contributor

Choose a reason for hiding this comment

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

What is the relevance if rdev?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Consult with @jimscarver for details abut the RDev reference.

Copy link
Contributor

Choose a reason for hiding this comment

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

@jimscarver ? What is the relevance of rdev?

@dckc
Copy link
Contributor

dckc commented Sep 9, 2021

This seems to be merged without peer review. What motivated that?

@Bill-Kunj
Copy link
Collaborator Author

Bill-Kunj commented Sep 10, 2021

In what way is this a fix? What was broken?

@dckc it's just a name for the branch - breaking it down semantically is unnecessary.

@Bill-Kunj
Copy link
Collaborator Author

This seems to be merged without peer review. What motivated that?

#240 (review) resulted in face-to-face review with other RGOV members. I merged it out of convenience and consideration for other people's time.

@dckc
Copy link
Contributor

dckc commented Sep 10, 2021

This seems to be merged without peer review. What motivated that?

#240 (review) resulted in face-to-face review with other RGOV members. I merged it out of convenience and consideration for other people's time.

OK, so there was peer review. Cool.

In the future, if you're acting on behalf of others, go ahead and make a note to leave an audit trail, please.

@jimscarver
Copy link
Contributor

jimscarver commented Sep 10, 2021 via email

@dckc
Copy link
Contributor

dckc commented Sep 10, 2021

the project is being performed by rdev decentralized governance. rdev is the testbed for rgov.

While rdev is as welcome as anyone to use rgov, rgov is not in any way controlled or owned or subsumed by rdev. The project is being performed by its contributors.

I thought I was pretty clear about this in the 2020-12-07 rdev meeting, though I evidently didn't manage to get it in the record.

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.

None yet

3 participants