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

feat: add navigation over Dockerfile-based projects #34

Merged
merged 1 commit into from
Nov 17, 2022

Conversation

ivanilves
Copy link
Owner

@ivanilves ivanilves commented Nov 15, 2022

This PR adds navigation over Dockerfile-based projects. As of release 0.3.X and later we use custom function to filter relevant project paths from the all existing Git repository paths. We pass a custom function to directory.Collect() as a parameter :neckbeard:

How to use?

Execute echo "mode: dockerfile" > .travelgrunt.yml in the root of a Dockerfile [mono]repo and then read this chapter.

The purpose of this PR is to demonstrate - how [easy] is to add a new working mode to travelgrunt - will it be our Dockerfile, Golang source files, Makefile, files with some custom content/comments or any other beasts you spot in typical wild [mono]repo, where directory tree depth and number of subfolders is way to big to just cd productivelly. #fatigue

We can also think about "true" monorepos with very complex structures and implement more YAML-driven configs like:

mode: advanced

rules:
  - path: terraform
    include:
      - terraform
      - terragrunt
  - path: code
    include:
      - golang
      - php
      - nodejs
   - path: build
     include: [Makefile]

Latter is for the future releases, of course, just thinking loudly here 😇

@codecov
Copy link

codecov bot commented Nov 15, 2022

Codecov Report

Merging #34 (3fa97e2) into main (8f95996) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #34   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            5         5           
  Lines          200       204    +4     
=========================================
+ Hits           200       204    +4     
Impacted Files Coverage Δ
pkg/config/config.go 100.00% <100.00%> (ø)
pkg/config/include/include.go 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@ivanilves ivanilves merged commit da9d7f6 into main Nov 17, 2022
@ivanilves ivanilves deleted the docker-include branch November 17, 2022 06:07
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