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

Modules section for dip #173

Merged
merged 1 commit into from
Jul 31, 2024
Merged

Modules section for dip #173

merged 1 commit into from
Jul 31, 2024

Conversation

emfy0
Copy link
Contributor

@emfy0 emfy0 commented Jul 30, 2024

Context

Managing group of projects with dip is hard, imagine having multiple gems with such setup

interaction:
  brakeman:
    description: Check brakeman sast
    command: docker run ...
  gitleaks:
    description: Check gitleaks sast
    command: docker run ...
  bearer:
    description: Check bearer sast
    command: docker run ...
  trivy:
    description: Check trivy sast
    command: docker run ...

Every change in sast configuration leads to change in all dip.yml among gems.

Related tickets

What's inside

This PR introduces new concept of modules, modules are stored in .dip folder. This folder should be synced among group for example with git submodules, so it is stored in one place and synced among group.

Each module contains dip configuration so it can be splitted logically, e.g.

# ./.dip/sasts.yml
interaction:
  brakeman:
    description: Check brakeman sast
    command: docker run ...
  gitleaks:
    description: Check gitleaks sast
    command: docker run ...
  bearer:
    description: Check bearer sast
    command: docker run ...
  trivy:
    description: Check trivy sast
    command: docker run ...

Now sasts are managed in one place and can be included via modules.

# ./dip.yml
modules:
  - sasts

interaction:
  my_special_task:
     ....

See more examples in README

Checklist:

  • I have added tests
  • I have made corresponding changes to the documentation

@bibendi
Copy link
Owner

bibendi commented Jul 31, 2024

Hey, Pavel! This is a great feature! Thank you for the contribution!

@bibendi bibendi merged commit 5a6c659 into bibendi:master Jul 31, 2024
5 checks passed
@bibendi
Copy link
Owner

bibendi commented Jul 31, 2024

I'll bump a new version today

@bibendi
Copy link
Owner

bibendi commented Aug 1, 2024

v8.1.0 has been released

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