Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Strip PHP 'v' version prefix #5

Merged
merged 1 commit into from
Jun 8, 2017
Merged

Strip PHP 'v' version prefix #5

merged 1 commit into from
Jun 8, 2017

Conversation

hmarr
Copy link
Contributor

@hmarr hmarr commented Jun 8, 2017

No description provided.

@hmarr hmarr requested a review from greysteil June 8, 2017 13:50
@hmarr hmarr merged commit ffe229b into master Jun 8, 2017
@hmarr hmarr deleted the php/strip-v-prefix branch June 8, 2017 13:55
deivid-rodriguez added a commit that referenced this pull request Jan 17, 2023
Pub has introduced a global `--color` flag, and we now get

```
/Users/deivid/Code/dependabot/dependabot-core/pub/lib/dependabot/pub/helpers.rb:206:in `block (3 levels) in run_dependency_services': dependency_services failed: Unhandled exception: (Dependabot::DependabotError)
Invalid argument(s): Could not find an option named "color".
#0      ArgResults.wasParsed (package:args/src/arg_results.dart:95)
#1      PubCommand._decideOnColors (package:pub/src/command.dart:305)
#2      PubCommand.run (package:pub/src/command.dart:182)
#3      CommandRunner.runCommand (package:args/command_runner.dart:209)
#4      _DependencyServicesCommandRunner.run (file:///Users/deivid/Code/dependabot/dependabot-core/pub/helpers/bin/dependency_services.dart:60)
#5      main (file:///Users/deivid/Code/dependabot/dependabot-core/pub/helpers/bin/dependency_services.dart:77)
#6      _delayEntrypointInvocation.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:295)
#7      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:192)
	from /Users/deivid/Code/dependabot/dependabot-core/pub/lib/dependabot/pub/helpers.rb:199:in `chdir'
	from /Users/deivid/Code/dependabot/dependabot-core/pub/lib/dependabot/pub/helpers.rb:199:in `block (2 levels) in run_dependency_services'
	from /Users/deivid/Code/dependabot/dependabot-core/common/lib/dependabot/shared_helpers.rb:168:in `with_git_configured'
	from /Users/deivid/Code/dependabot/dependabot-core/pub/lib/dependabot/pub/helpers.rb:189:in `block in run_dependency_services'
	from /Users/deivid/Code/dependabot/dependabot-core/common/lib/dependabot/shared_helpers.rb:49:in `block in in_a_temporary_directory'
	from /Users/deivid/Code/dependabot/dependabot-core/common/lib/dependabot/shared_helpers.rb:49:in `chdir'
	from /Users/deivid/Code/dependabot/dependabot-core/common/lib/dependabot/shared_helpers.rb:49:in `in_a_temporary_directory'
	from /Users/deivid/Code/dependabot/dependabot-core/pub/lib/dependabot/pub/helpers.rb:182:in `run_dependency_services'
	from /Users/deivid/Code/dependabot/dependabot-core/pub/lib/dependabot/pub/helpers.rb:36:in `dependency_services_list'
	from /Users/deivid/Code/dependabot/dependabot-core/pub/lib/dependabot/pub/file_parser.rb:30:in `list'
	from /Users/deivid/Code/dependabot/dependabot-core/pub/lib/dependabot/pub/file_parser.rb:17:in `parse'
	from bin/dry-run.rb:553:in `block in <main>'
	from bin/dry-run.rb:286:in `cached_read'
	from bin/dry-run.rb:553:in `<main>'
```

when delegating to Pub.

This change seems to address that.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants