Skip to content

Conversation

@mand35
Copy link
Contributor

@mand35 mand35 commented Apr 9, 2019

You now named the variable exclusive_access, and not just exclusive anymore. This needs to be reflected in the documentation

You now named the variable exclusive_access, and not just exclusive anymore. This needs to be reflected in the documentation
@jenkins-cscs
Copy link
Collaborator

Can I test this patch?

@codecov-io
Copy link

codecov-io commented Apr 9, 2019

Codecov Report

Merging #742 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #742   +/-   ##
=======================================
  Coverage   91.73%   91.73%           
=======================================
  Files          77       77           
  Lines        9527     9527           
=======================================
  Hits         8740     8740           
  Misses        787      787

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 2c41d15...3fddba9. Read the comment docs.

@mand35
Copy link
Contributor Author

mand35 commented Apr 9, 2019 via email

@vkarak vkarak self-requested a review April 9, 2019 07:05
@vkarak vkarak changed the title tutorial: exclusive access [doc] Fix reference to exclusive_access in tutorial Apr 9, 2019
@vkarak vkarak added this to the ReFrame sprint 2019w13 milestone Apr 9, 2019
@vkarak
Copy link
Contributor

vkarak commented Apr 9, 2019

@mand35 Thanks for spotting this! It has always been exclusive_access the variable, so this bug has been existing forever :-)

@vkarak vkarak merged commit 3724185 into reframe-hpc:master Apr 9, 2019
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.

4 participants