-
Notifications
You must be signed in to change notification settings - Fork 8
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
format templates to follow Drupal practices #123
Conversation
This is really good, thanks @markconroy. |
@andybroomfield I've changed one test where I could see the class needed to change, but tests seem to be still breaking. Could you fix that by any chance? |
Sometimes we have to re-run the tests if the tests have been changed, maybe something to do with paratest? @stephen-cox |
Looks like it passed. @andybroomfield if you're okay with this now can I leave it to you to merge this for the release? If anyone is using localgov_theme, they will need to update their CSS classes - I think the only one might be changing Thanks a lot. I'm really happy with how this module has turned out. Great work. |
Closes #122