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

Fix specialist_sector_cleanup task. #2270

Merged
merged 1 commit into from
Jul 14, 2015
Merged

Conversation

alext
Copy link
Contributor

@alext alext commented Jul 13, 2015

This was blowing up if it encountered an orphaned tagging. This updates
it to be more robust and handle this case.

alext added a commit to alphagov/govuk-lint that referenced this pull request Jul 13, 2015
There are times when it's appropriate to use single quotes.
Specifically when maintaining consistency within an area of the
codebase (this linter it typically only applied to the diffs, and
therefore won't take into account existing uses of single-quotes - eg
alphagov/whitehall#2270).  It therefore doesn't
make sense to enforce this automatically.
This was blowing up if it encountered an orphaned tagging.  This updates
it to be more robust and handle this case.
@alext alext force-pushed the fix_specialist_sector_cleanup_task branch from 909afc8 to 3e40046 Compare July 13, 2015 10:53
@rboulton
Copy link
Contributor

👍

alext added a commit that referenced this pull request Jul 14, 2015
…task

Fix specialist_sector_cleanup task.
@alext alext merged commit 455fe03 into master Jul 14, 2015
@alext alext deleted the fix_specialist_sector_cleanup_task branch July 14, 2015 08:41
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