-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
Permalink
Choose a base ref
{{ refName }}
default
Choose a head ref
{{ refName }}
default
Comparing changes
Choose two branches to see what’s changed or to start a new pull request.
If you need to, you can also or
learn more about diff comparisons.
Open a pull request
Create a new pull request by comparing changes across two branches. If you need to, you can also .
Learn more about diff comparisons here.
base repository: symfony/symfony
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: a003380
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
...
head repository: symfony/symfony
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 17ad6fd
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
- 8 commits
- 3 files changed
- 4 contributors
Commits on Mar 14, 2015
-
Configuration menu - View commit details
-
Copy full SHA for bc066fb - Browse repository at this point
Copy the full SHA bc066fbView commit details -
Configuration menu - View commit details
-
Copy full SHA for 3ae52ed - Browse repository at this point
Copy the full SHA 3ae52edView commit details
Commits on Mar 15, 2015
-
Configuration menu - View commit details
-
Copy full SHA for b8d42b3 - Browse repository at this point
Copy the full SHA b8d42b3View commit details -
bug #13927 Fixing wrong variable name from #13519 (weaverryan)
This PR was merged into the 2.3 branch. Discussion ---------- Fixing wrong variable name from #13519 | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #13519 | License | MIT | Doc PR | n/a Hi guys! I'm currently seeing an undefined variable in the `PhpDumper` on all branches. I think it was added inadvertently in #13519, so unless I'm totally missing something, this should be an easy merge. Thanks! Commits ------- 3ae52ed Fixing wrong variable name from #13519
Configuration menu - View commit details
-
Copy full SHA for 225dbaf - Browse repository at this point
Copy the full SHA 225dbafView commit details -
minor #13930 [DependencyInjection]
ResolveParameterPlaceHoldersPass
…… unit tests (xabbuh) This PR was merged into the 2.3 branch. Discussion ---------- [DependencyInjection] `ResolveParameterPlaceHoldersPass` unit tests | Q | A | ------------- | --- | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | License | MIT | Doc PR | When working #13924, I noticed that there were no tests for the `ResolveParameterPlaceHoldersPass` class. So I decided to add them for the `2.3` branch too. Commits ------- b8d42b3 `ResolveParameterPlaceHoldersPass` unit tests
Configuration menu - View commit details
-
Copy full SHA for d50f0a8 - Browse repository at this point
Copy the full SHA d50f0a8View commit details -
minor #13925 [translation][initialize cache] Remove dead code. (aitbo…
…udad) This PR was merged into the 2.6 branch. Discussion ---------- [translation][initialize cache] Remove dead code. | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Fixed tickets | ~ | Tests pass? | yes | License | MIT I see two issues here: 1- initialize function doesn't exist in Translator class. 2- initializeCacheCatalogue is only called when cache is not null Commits ------- bc066fb [translation][initialize cache] Remove dead code.
Configuration menu - View commit details
-
Copy full SHA for c923613 - Browse repository at this point
Copy the full SHA c923613View commit details -
* 2.3: `ResolveParameterPlaceHoldersPass` unit tests Fixing wrong variable name from #13519
Configuration menu - View commit details
-
Copy full SHA for be05344 - Browse repository at this point
Copy the full SHA be05344View commit details -
* 2.6: `ResolveParameterPlaceHoldersPass` unit tests Fixing wrong variable name from #13519 [translation][initialize cache] Remove dead code. Conflicts: src/Symfony/Component/DependencyInjection/Dumper/PhpDumper.php
Configuration menu - View commit details
-
Copy full SHA for 17ad6fd - Browse repository at this point
Copy the full SHA 17ad6fdView commit details
Loading
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff a003380...17ad6fd