QuickFeed: Instant Feedback on Programming Assignments

Documentation

• Teachers that wants to use QuickFeed may wish to review the User Manual.
• Teachers may also want to copy the sign up instructions and lab submission instructions, and make the necessary adjustments for your course.
• Installation instructions for QuickFeed.

Contributing

The following instructions assume you have installed the GitHub CLI. See here for installation instructions for your platform.

On systems with homebrew:

% brew install gh
% gh help

Create Issue First

Before you implement some feature or bug fix, you should open an issue first. This issue should then be linked in the corresponding pull request.

Create Pull Request

Before starting a new pull request, either clone the repo:

% gh repo clone quickfeed/quickfeed
% cd quickfeed

Or if you have already cloned, make sure to start from an up-to-date master branch:

# Make sure to start from master branch
% git checkout master
# Make sure your master branch is up-to-date
% git pull

To create a pull request on the main repository follow these steps.

# Create and switch to your new feature branch
% git switch -C <feature-branch>
# Edit and stage files
% git add <files>
% git commit
# When done and ready to share
% gh pr create --title "Short description of the feature or fix"
# Alternatively: Use --draft if you want to share your code, but want to continue developing
% gh pr create --draft --title "Short description of the feature or fix"

To continue development on a pull request (same branch as before):

# Only necessary if you previously switched away from the feature-branch
% git switch <feature-branch>
# Edit and stage files
% git add <files>
% git commit
% git push

To fetch an existing pull request to your local machine.

% gh pr checkout <PR#>

For additional details on the gh pr and gh pr create commands:

% gh help pr
% gh help pr create

GitHub Issues and Pull Requests

When creating a pull request, it is always nice to connect it to a GitHub issue describing the feature or problem you are fixing. If there is an issue that is fixed by your pull request please remember to add one of the following lines at the end of the pull request description.

Closes <Issue#>.
Fixes <Issue#>.
Resolves <Issue#>.

For detailed instructions on configuring QuickFeed for development, please see our Developer Guide.

Style Guidelines

We chose to implement QuickFeed in Go and Typescript because these languages offer simplicity and type safety. We therefore require that certain style guidelines are followed when creating pull requests.

For Go, we expect code to follow these style guidelines and list of common mistakes:

• We use the golangci-lint linter in VSCode.

• Go Code Review Comments. In particular, note the section on how to Handle Errors, Mixed Caps, Variable Names.

For Typescript, we think these style guidelines look reasonable. Moreover, the formatOnSave and tslint.run options in VSCode should help maintain reasonable style.

Note that we currently violate the interface naming guideline by using the I prefix on some interfaces, and several of the other guidelines. We have started to rename these interfaces, and will eventually rename all such interfaces.