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

ADD sidecar API support to (force) push changes without committing. #389

Merged
merged 2 commits into from
Sep 12, 2022

Conversation

PIG208
Copy link
Contributor

@PIG208 PIG208 commented Sep 12, 2022

This allows the student to force push or push without generating a commit
via the CLI without using their own credentials. Making git push and
git push -f available should be sufficient for students to maneuver with
Git on our IDE instances for the purposes of learning VCS.

It's unlikely that we are going to support more arguments without going into the depth
of actually figuring out a way to keep students authenticated and use git directly.

Allowing the students to push via Anubis CLI is only for the convenience
of not requiring them to enter a GitHub token everytime they push or
storing their SSH keys on the server if they decide to use Git in the IDE.

This is manually tested locally by invoking the added API via the CLI.

With this new option, we can push without generating a commit via the CLI.

Signed-off-by: Zixuan James Li <p359101898@gmail.com>
This allows the student to force push via the CLI. Making `git push` and
`git push -f` available should be sufficient for students to maneuver with
Git on our IDE instances for the purposes of learning. It's unlikely
that we are going to support more arguments.

Allowing the students to push via Anubis CLI is only for the convenience
of not requiring them to set up their own authentication mechanism on
GitHub or storing their SSH keys or GitHub token on the server.

Signed-off-by: Zixuan James Li <p359101898@gmail.com>
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