Skip to content

Commit

Permalink
Bump version
Browse files Browse the repository at this point in the history
  • Loading branch information
gregbell26 committed Jun 29, 2024
1 parent 23e9cf9 commit 2f73225
Show file tree
Hide file tree
Showing 6 changed files with 743 additions and 746 deletions.
2 changes: 1 addition & 1 deletion .github/ci.yaml → .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -61,4 +61,4 @@ jobs:

- name: Print Output
id: output
run: echo "${{ steps.test-action.outputs.time }}"
run: echo "${{ steps.test-action.outputs.time }}"
23 changes: 10 additions & 13 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,15 +2,14 @@

This action deploys autograders to Gradescope using the default zip file upload for the ubuntu base image.

Per the terms of service, usage of a bot similar to this one is permissible as it acts as a human would when accessing
Per the terms of service, usage of a bot similar to this one is permissible as it acts as a human would when accessing
the service. https://www.gradescope.com/tos.
Please refer to the terms of service for Gradescope prior to using this.
By using this, you are accepting the inherent risks of using automated ui tools on Gradescope.

This may stop working at any time (especially if Gradescope change its UI / obscuration techniques).


This should be run by a separate 'service user' that has TA permissions **only** for the class that this workflow is
This should be run by a separate 'service user' that has TA permissions **only** for the class that this workflow is
deployed for.

SSO accounts are not currently supported, and it is unlikely that they will be.
Expand All @@ -19,26 +18,25 @@ If Gradescope implements MFA (which they should), this will stop working.

This workflow does not attempt to access any scores or grading information and does not store any data.

If you feel dubious about that claim, please check out the code and inspect it yourself.
If you feel dubious about that claim, please check out the code and inspect it yourself.
It is under 200 lines, so it should be easy to verify.

Under the hood, puppeteer (a JavaScript UI testing framework), is used to interact with Gradescope.


## Inputs

### `course_id`

**Required** Set the course id for which this action is deployed.
**Required** Set the course id for which this action is deployed.

Can be found in the URI after 'courses'
Can be found in the URI after 'courses'

This should be set as a repository variable.
This should be set as a repository variable.
It is highly recommended **not** to hard code this in the workflow.

### `gradescope_username`

**Required** Set the gradescope service user's username.
**Required** Set the gradescope service user's username.

This should be set as a repository secret.

Expand All @@ -58,7 +56,7 @@ For the love of all things holy, store this as a secret.

### `gradescope_assignment_id`

**Required** Set the gradescope assignment id.
**Required** Set the gradescope assignment id.

This should be derived from the artifacts produced during the CI pipeline

Expand All @@ -68,7 +66,7 @@ This should be derived from the artifacts produced during the CI pipeline

This should be derived from the artifacts produced during the CI pipeline

If running as a separate job (which it probably should be), you may need to run the `actions/download-artifact` action
If running as a separate job (which it probably should be), you may need to run the `actions/download-artifact` action
to get pull it into this job.

## Outputs
Expand All @@ -89,5 +87,4 @@ with:
gradescope_password: ${{ secrets.GRADESCOPE_SERVICE_USER_PASSWORD }}
gradescope_assignment_id: ${{ matrix.assignments.assignment_id }}
artifact_path: "${{ github.workspace }}/${{ matrix.assignments.assignment_path }}"

```
```
3 changes: 1 addition & 2 deletions action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@ outputs:
status:
description: "The workflow status."


runs:
main: dist/index.js
using: node20
using: node20
Loading

0 comments on commit 2f73225

Please sign in to comment.