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

Added commit from internal repo, housekeeping. #2

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

mparker17
Copy link

Problem / motivation

There was a commit on our internal repo (45a7fe1) that had not been pushed here but was being used in our environments, so I pushed it to CWRC's fork.

I also:

  • Made the module conform to Drupal coding and documentation standards.
  • Removed unused variables.
  • Organized .module file, grouping together related functions.
  • Added an empty gitignore (Github recommends every repo have one of these even if they are empty).
  • Added Github issue and pull request templates (I used the PR template for this pull request!)
  • Added Contributing Guidelines (Github recommends every repo have one of these)

Proposed resolution

Merge in the changes on CWRC's master branch (i.e.: this pull request)

Remaining tasks

  • Code review
  • Commit

User interface changes

None.

API changes

None.

Data model changes

None.

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.

1 participant