Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Migrate locks #157

Merged
merged 3 commits into from
Mar 15, 2019
Merged

Migrate locks #157

merged 3 commits into from
Mar 15, 2019

Conversation

OdysseusLives
Copy link
Member

Migrate to core lock format via the new migrateToCoreLocks task

This will lock standard configurations, as well as those from facets such as those created via the nebula-project-plugin or others whose family of configuration names contain one with CompileOnly in the configuration name (smokeTestCompileOnly, for example)

You can also lock all configurations via the property lockAllConfigurations if that's what is wanted

@@ -0,0 +1,76 @@
/**
*
* Copyright 2018 Netflix, Inc.
Copy link
Member

Choose a reason for hiding this comment

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

not a big deal but this might have to be 2019 :)

Copy link
Member Author

Choose a reason for hiding this comment

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

Haha, yes! I'll adjust that. Thanks!

List<String> findConfigurationsToLock() {

def configurationsToLock = new ArrayList<>()
def baseConfigurations = [
Copy link
Member

Choose a reason for hiding this comment

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

nice!!

@coveralls
Copy link

coveralls commented Mar 15, 2019

Coverage Status

Coverage decreased (-0.9%) to 87.33% when pulling 0a34c25 on migrate_locks into 87dd3da on master.

import org.gradle.api.tasks.OutputDirectory
import org.gradle.api.tasks.TaskAction

class MigrateToCoreLocksTask extends AbstractLockTask {
Copy link
Member

Choose a reason for hiding this comment

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

This is awesome! thanks for adding a task!

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm glad for it, too!

Copy link
Member

@rpalcolea rpalcolea left a comment

Choose a reason for hiding this comment

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

This is super cool!

@OdysseusLives OdysseusLives merged commit cf6df6e into master Mar 15, 2019
@OdysseusLives OdysseusLives deleted the migrate_locks branch March 15, 2019 22:18
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.

None yet

3 participants