Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

PLAT-47 improv #59

Merged
merged 4 commits into from
Feb 26, 2016
Merged

PLAT-47 improv #59

merged 4 commits into from
Feb 26, 2016

Conversation

Saphyel
Copy link
Contributor

@Saphyel Saphyel commented Feb 26, 2016

Fixes https://jira.comicrelief.com/browse/PLAT-47

Changes proposed in this pull request

  • Fix make.yml
  • Fix filter....yml
  • Fix phing: build.xml

Deployment instructions

  • No manual actions required for deployment

@Saphyel Saphyel changed the title Feature/plat 47 improv PLAT-47 improv Feb 26, 2016
@pvhee
Copy link
Contributor

pvhee commented Feb 26, 2016

@Saphyel do you think it would make sense as well to make phing config:export run on travis as well, and if the output is dirty (e.g. there are new file changes in the git repo), then let it fail?

@Saphyel
Copy link
Contributor Author

Saphyel commented Feb 26, 2016

@pvhee to be honest make no sense, the filter.format.yml is because the cr.install overwrite it (the code is execute after the config/install finish)... and what we develop in the future I hope should not be overwritten 😨 (if not what is the point of the yml files?)

So if the question is could be useful? Yes
if the question is should happen? No

@pvhee
Copy link
Contributor

pvhee commented Feb 26, 2016

@Saphyel can we maybe then remove that line from our .info.yml so it won't be re-exported using config:export?

@Saphyel
Copy link
Contributor Author

Saphyel commented Feb 26, 2016

if we change them in the future we'll have to add them back, agov have them: https://github.com/previousnext/agov/tree/8.x-1.x/agov/config/install
Maybe we should remove the code that changes the filter.format

@pvhee
Copy link
Contributor

pvhee commented Feb 26, 2016

@Saphyel could you research what is best to do here? And with agov, if you run config:export, do you also have it overwritten after install?

@Saphyel
Copy link
Contributor Author

Saphyel commented Feb 26, 2016

@pvhee could we merge this and choose later how to deal with the filter formats? I don't think they gonna be a problem in short term, and the other stuff is important to merge before someone run phing make

@pvhee
Copy link
Contributor

pvhee commented Feb 26, 2016

@Saphyel definitely - merged in

pvhee added a commit that referenced this pull request Feb 26, 2016
@pvhee pvhee merged commit 716dd35 into develop Feb 26, 2016
@pvhee pvhee deleted the feature/PLAT-47_improv branch February 26, 2016 16:39
@pvhee pvhee mentioned this pull request Jan 5, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants