Skip to content
This repository was archived by the owner on May 17, 2019. It is now read-only.

Commit f31332d

Browse files
authored
Merge pull request #21 from NoRedInk/fix-other-package-version-in-change-message
elm_deps_sync: Log message before updating other_package_version
2 parents e7b38f8 + e2d07af commit f31332d

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

elm_deps_sync.py

+3-3
Original file line numberDiff line numberDiff line change
@@ -28,12 +28,12 @@ def sync_versions(top_level_file, spec_file, quiet=False, dry=False, note_test_d
2828
package_name=package_name, spec_file=spec_file, package_version=package_version)
2929
)
3030
elif spec['dependencies'][package_name] != package_version:
31-
spec['dependencies'][package_name] = package_version
32-
3331
messages.append('Changing {package_name} from version {package_version} to {other_package_version}'.format(
3432
package_version=package_version, package_name=package_name,
3533
other_package_version=spec['dependencies'][package_name])
36-
)
34+
)
35+
36+
spec['dependencies'][package_name] = package_version
3737

3838
test_deps = {}
3939

0 commit comments

Comments
 (0)