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 #2658: Run invalidateContainer() only when needed #2663

Merged

Conversation

OnkelTem
Copy link
Contributor

@OnkelTem OnkelTem commented Mar 9, 2017

No description provided.

@OnkelTem
Copy link
Contributor Author

OnkelTem commented Mar 9, 2017

Sorry I'm not a big expert in running tests, so haven't launched them locally :-/

@greg-1-anderson
Copy link
Member

Only one 'make' test failing here. Probably a spurious failure; I restarted that one test.

@greg-1-anderson
Copy link
Member

Failed the same way -- although I can download the file on my local system just fine.

Note that make has been remove from the master branch. Please always start with all PRs against master, and then backport to 8.x once merged on master.

@weitzman
Copy link
Member

I re-ran the last failed job and now the PR is green. Unless @greg-1-anderson objects in the next day or two, I'm inclined to merge this.

@weitzman weitzman changed the title Github Issue #2658: Run invalidateContainer() only when needed Fix #2658: Run invalidateContainer() only when needed Mar 10, 2017
@greg-1-anderson
Copy link
Member

Figured it would eventually turn green.

I am strongly in favor of this change. We should do the same on the master branch as well. Code should be very similar.

@weitzman weitzman merged commit c6191e7 into drush-ops:8.x Mar 10, 2017
weitzman pushed a commit that referenced this pull request Mar 10, 2017
# Conflicts:
#	lib/Drush/Drupal/DrushServiceModfier.php
#	lib/Drush/Drupal/DrushServiceModifier.php
#	src/Drupal/DrushServiceModfier.php
mikeker pushed a commit to mikeker/drush that referenced this pull request Aug 10, 2017
…-ops#2663)

# Conflicts:
#	lib/Drush/Drupal/DrushServiceModfier.php
#	lib/Drush/Drupal/DrushServiceModifier.php
#	src/Drupal/DrushServiceModfier.php
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.

3 participants