Skip to content

(maint) Update sync.yaml for new modsync config options#208

Merged
michaeltlombardi merged 2 commits intopuppetlabs:masterfrom
glennsarti:update-sync.yml
Feb 12, 2018
Merged

(maint) Update sync.yaml for new modsync config options#208
michaeltlombardi merged 2 commits intopuppetlabs:masterfrom
glennsarti:update-sync.yml

Conversation

@glennsarti
Copy link
Copy Markdown
Contributor

Previously the travis.yaml file was unmanaged as it contained customisations not
compatible with modsync. Now that the required options are available the sync
file can be updated accordingly. This commit:

  • Updates the sudo, dist and before_install commands to install Powershell
  • Updates the NOTICE file to indicate the correct dates for the copyright
    holders

Previously the travis.yaml file was unmanaged as it contained customisations not
compatible with modsync.  Now that the required options are available the sync
file can be updated accordingly.  This commit:
- Updates the sudo, dist and before_install commands to install Powershell
- Updates the NOTICE file to indicate the correct dates for the copyright
  holders
@glennsarti
Copy link
Copy Markdown
Contributor Author

Travis.yaml is to remain unmanaged until puppetlabs/modulesync_configs#178 is merged

@michaeltlombardi
Copy link
Copy Markdown
Contributor

@glennsarti changes look fine, puppetlabs/modulesync_configs#178 is merged

@glennsarti glennsarti changed the title {DO NOT MERGE} (maint) Update sync.yaml for new modsync config options (maint) Update sync.yaml for new modsync config options Jan 14, 2018
This commit applies modsync at commit a3ae5c94f7d
Comment thread .sync.yml
platforms: ["mswin", "mingw", "x64_mingw"]
version: '< 2.6.0'
.travis.yml:
unmanaged: true
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So this file's now unmanaged but we're adding settings for it?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correct. We are now managing this file with the specified settings.

@michaeltlombardi michaeltlombardi merged commit e53461c into puppetlabs:master Feb 12, 2018
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.

2 participants