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

WIP chmod #143

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

WIP chmod #143

wants to merge 3 commits into from

Conversation

TheFern2
Copy link

@TheFern2 TheFern2 commented Mar 21, 2021

This is a work in progress PR. Might take a while, as I am doing this mainly to learn rust.

  • Copy .template
  • Replace "template" occurrences in build.rs by the TOOL_NAME.
  • Make changes in Cargo.toml indicated fields.
  • Configure argument parsing with clap at cli.rs, here are some rules
  • Every help message should start with upper case letter and end with a dot.
  • Exit code is defined by the POSIX standard (extra customization might be required for extensions).
  • Update the README with the new status of the tool and contributors section.
  • Unit tests passed
  • Tool completed for review

@TheFern2
Copy link
Author

@GrayJack just wanted to check repo still alive? Also I see chmod was already on the roadmap, so not sure if anyone is already working on it.

@GrayJack
Copy link
Owner

Yeah, this project is alive, the roadmap is kinda a mess right now, I'll fix it as soon as I have some free time, but the chmod is open to be implemented!

I think in the issue I did put what is required for the issue to be closed, but we're open to partial implementations PR.

Any questions, feel free to ask.

@TheFern2
Copy link
Author

Yeah, this project is alive, the roadmap is kinda a mess right now, I'll fix it as soon as I have some free time, but the chmod is open to be implemented!

I think in the issue I did put what is required for the issue to be closed, but we're open to partial implementations PR.

Any questions, feel free to ask.

Awesome, I will prob do a full implementation. Just wanted to check before diving in. Cheers!

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