Skip to content

[bugfix] Restrict use of required keyword#2012

Merged
jjotero merged 6 commits intoreframe-hpc:masterfrom
jjotero:bugfix/use-require-keyword
Jun 25, 2021
Merged

[bugfix] Restrict use of required keyword#2012
jjotero merged 6 commits intoreframe-hpc:masterfrom
jjotero:bugfix/use-require-keyword

Conversation

@jjotero
Copy link
Copy Markdown
Contributor

@jjotero jjotero commented Jun 11, 2021

See linked issue for details.

Closes #2011.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jun 11, 2021

Codecov Report

Merging #2012 (5631c9f) into master (830abc6) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2012   +/-   ##
=======================================
  Coverage   87.48%   87.48%           
=======================================
  Files          50       50           
  Lines        8851     8853    +2     
=======================================
+ Hits         7743     7745    +2     
  Misses       1108     1108           
Impacted Files Coverage Δ
reframe/core/meta.py 98.80% <ø> (ø)
reframe/core/variables.py 96.06% <100.00%> (+0.02%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 830abc6...5631c9f. Read the comment docs.

@jjotero
Copy link
Copy Markdown
Contributor Author

jjotero commented Jun 24, 2021

@jenkins-cscs retry dom

@jjotero
Copy link
Copy Markdown
Contributor Author

jjotero commented Jun 25, 2021

@jenkins-cscs retry tsa

@jjotero jjotero merged commit cca262b into reframe-hpc:master Jun 25, 2021
@jjotero jjotero deleted the bugfix/use-require-keyword branch June 25, 2021 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make illegal to set a regular Python class attribute as required

4 participants