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

Rename iteration to submission #161

Merged
merged 5 commits into from
Oct 6, 2020
Merged

Conversation

iHiD
Copy link
Member

@iHiD iHiD commented Oct 6, 2020

This PR does not need reviewing (else you'll be here all day)


This PR renames iterations to submissions.

  • All code submitted will now be in stored as a submission.
  • Submissions are a bundle of files, that have the tests run on them. They are what is submitted via the CLI etc.
  • Submissions are essentially an internal construct - users wouldn't go back and look at previous submissions. It is expected most will fail the tests.
  • Iterations are effectively breakpoints that we show to users. A future PR will add them in.

Comments etc will still be stored on iterations, not on submissions.

@iHiD iHiD changed the title Rename iteration to submission [WIP] Rename iteration to submission Oct 6, 2020
@iHiD iHiD force-pushed the rename-iteration-to-submission branch from 13c2091 to 1ba9143 Compare October 6, 2020 20:59
@iHiD iHiD changed the title [WIP] Rename iteration to submission Rename iteration to submission Oct 6, 2020
@@ -21,7 +21,7 @@ gem 'rest-client' # http
gem 'mandate', '0.4.0.beta.1'

# Setup dependencies
gem 'exercism-config', '>= 0.43.0'
gem 'exercism-config', '>= 0.44.0'
Copy link
Member Author

Choose a reason for hiding this comment

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

This changes the config so the field is now aws_submissions_bucket not aws_iterations_bucket

@@ -5,8 +5,9 @@ ruby '2.6.6'
# Service/framework dependencies
gem 'rails',
git: 'http://github.com/rails/rails.git',
ref: '7101489293ece071013417aeed2da3aa1b0927c9'
ref: '207f116234374d6414593246240b05261a96fa3a'
Copy link
Member Author

Choose a reason for hiding this comment

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

Sporadic bumping of Rails that I thought I'd do while breaking everything else at the same time 😉

@@ -1,3 +1,5 @@
class Delayed::Job
self.abstract_class = true
Copy link
Member Author

Choose a reason for hiding this comment

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

Needed because of rails/rails#40219

@iHiD iHiD force-pushed the rename-iteration-to-submission branch from 1ba9143 to c82ee24 Compare October 6, 2020 21:09
@iHiD iHiD marked this pull request as ready for review October 6, 2020 21:09
@iHiD
Copy link
Member Author

iHiD commented Oct 6, 2020

I'm going to merge this, because everything else should be rebased onto it, and doing so should be straightforward. But if it gets more out of sync it'll become impossible 🙂

@iHiD iHiD force-pushed the rename-iteration-to-submission branch from c82ee24 to 62d1e9a Compare October 6, 2020 22:03
@iHiD iHiD merged commit 973059b into master Oct 6, 2020
@iHiD iHiD deleted the rename-iteration-to-submission branch October 6, 2020 23:46
@ErikSchierboom
Copy link
Member

👍

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