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

Update module-modules-sea.R #6

Merged
merged 1 commit into from
Apr 30, 2019
Merged

Conversation

e-leib
Copy link
Contributor

@e-leib e-leib commented Apr 29, 2019

Updated the variable name references to reflect new column name and levels in load-sea-constants-modules.R. Now congruency column (formerly known as num_larger).

Updated the variable name references to reflect new column name and levels in load-sea-constants-modules.R. Now congruency column (formerly known as num_larger).
@monicathieu
Copy link
Collaborator

Ok, these changes look good. I will approve and merge both pull requests!

Note that, in the future, you should be able to (method is slightly different depending on where you init your pull requests from) add subsequent commits back onto the same branch you are PR-ing from so you can have just one PR with multiple changes on it (easier for everyone to track!)

@monicathieu monicathieu merged commit 2e1f5fe into joaquinanguera:master Apr 30, 2019
@e-leib
Copy link
Contributor Author

e-leib commented Apr 30, 2019 via email

@e-leib e-leib deleted the patch-1 branch May 5, 2019 18:07
@e-leib e-leib restored the patch-1 branch May 5, 2019 18:07
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