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

Add AWS Codecommit support #159

Closed

Conversation

davewongillies
Copy link
Contributor

AWS Codecommit doesn't have either .git at the end of the url nor the concept of namespaces. For example, codecommit urls are in the form of {ssh,https}://git-codecommit.<aws_region>.amazonaws.com/v1/repos/<reponame>

Copy link
Contributor

@joestump joestump left a comment

Choose a reason for hiding this comment

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

Given that this works with config changes to modulesync.yml, I'd vote this be refactored into a documentation change.

@@ -105,6 +105,10 @@ def self.manage_module(puppet_module, module_files, module_options, defaults, op
namespace, module_name = module_name(puppet_module, options[:namespace])
unless options[:offline]
git_base = options[:git_base]
if git_base.include? 'git-codecommit'
namespace = 'v1/repos'
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not just document namespace needs to be set to this for CodeCommit?

README.md Show resolved Hide resolved
lib/modulesync/git.rb Outdated Show resolved Hide resolved
@bastelfreak
Copy link
Member

Hey @davewongillies, thanks for the PR! Can you take a look at the failing travis tests?

@ekohl
Copy link
Member

ekohl commented Dec 21, 2020

Is there any interest to pick this up again or shall we close it for inactivity?

@ekohl
Copy link
Member

ekohl commented Apr 22, 2021

I'm closing this for inactivity. I also agree with #159 (review) that this should be a documentation only example.

@ekohl ekohl closed this Apr 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants