profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/MaxymVlasov/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
Maksym Vlasov MaxymVlasov Star Kyiv, Ukraine https://t.me/catops CatOps Engineer. OSS enthusiast. Speaker - https://u.to/tBrTGA. Co-founder @itkpi. Microsoft Student Partner

antonbabenko/pre-commit-terraform 1244

pre-commit git hooks to take care of Terraform configurations

itkpi/docker-utils 2

Utility scripts for docker images

MaxymVlasov/eco-data-visualizer 2

Data visualizer + AQI calculator for CSV-files provided by SaveEcoBot

MaxymVlasov/iothub-to-insight 1

Send data from Azure IoTHub to Application Insight

anastyakulyk/community 0

Kubernetes community content

MaxymVlasov/aws_products 0

Script to retrieve the list of AWS Services and their one-line descriptions

startedstakater/Reloader

started time in 19 hours

PullRequestReviewEvent

issue openedantonbabenko/pre-commit-terraform

fix: pre-commit PR check. Add shfmt

https://github.com/antonbabenko/pre-commit-terraform/runs/3632045056

Relates: #229

created time in 6 days

issue commentantonbabenko/pre-commit-terraform

terraform_docs: unable to specify relative PATH to config file

That really looks like a bug. Depends on #155 merge.

  • reminder: Need to document TFlint relative path usage

Thank you, @7adityaraj ! I will fix this in ~3-4 weeks. If it too long - you can try prepare fix based on #155 and send PR (don't forget mention here if you'll start contibuting)

7adityaraj

comment created time in 10 days

issue commentantonbabenko/pre-commit-terraform

unable to pass config file from pre-commit

Is it work if use tag 1.50 from this repo?

repos:
- repo: https://github.com/antonbabenko/pre-commit-terraform 
  rev: v1.50.0 
  hooks: 
  - id: terraform_docs
    args:
    - --config=__GIT_WORKING_DIR__/precommit/.terraform-docs.yml
    - md
7adityaraj

comment created time in 10 days

issue commentantonbabenko/pre-commit-terraform

unable to pass config file from pre-commit

Like this one Screenshot_20210915-002933.png

7adityaraj

comment created time in 10 days

issue commentantonbabenko/pre-commit-terraform

unable to pass config file from pre-commit

And __GIT_WORKING_DIR__ - is not full path, it argument that should be passead as is

@7adityaraj

7adityaraj

comment created time in 10 days

issue commentantonbabenko/pre-commit-terraform

unable to pass config file from pre-commit

This is exepted behavior.

I will clarify docs about that.

7adityaraj

comment created time in 10 days

issue commentantonbabenko/pre-commit-terraform

unable to pass config file from pre-commit

Good, at least now it show right path to config file in error.

@7adityaraj , using this fork, try change first arg to

- --config=__GIT_WORKING_DIR__/precommit/.terraform-docs.yml
7adityaraj

comment created time in 10 days

issue commentantonbabenko/pre-commit-terraform

unable to pass config file from pre-commit

Hi, @7adityaraj . I think it can be fixed by #155. You can try clone repo, checkout to PR branch, and use pre-commit try-repo (more details) to check is it works for you.

After, check, please describe is it help you.

7adityaraj

comment created time in 10 days

push eventantonbabenko/pre-commit-terraform

Maksym Vlasov

commit sha e7a0b50fe4b7a777b89d69ef9f9aec28fbbec3e0

Update bug_report_local_install.md

view details

push time in 10 days

push eventantonbabenko/pre-commit-terraform

Maksym Vlasov

commit sha 285d12f2e1d5680309b332dd3e4fd22e57080df8

Update bug_report_docker.md

view details

push time in 10 days

push eventantonbabenko/pre-commit-terraform

Maksym Vlasov

commit sha b5eae47bcbebc6f3fb85b82f21f9dcf04a64cc8f

Update and rename bug_report_local_install--.md to bug_report_local_install.md

view details

push time in 10 days

push eventantonbabenko/pre-commit-terraform

Maksym Vlasov

commit sha da171e53e5a62904daae0135ea725761667ff5b9

Update and rename bug_report_local_install.md to bug_report_local_install--.md

view details

push time in 10 days

push eventantonbabenko/pre-commit-terraform

Maksym Vlasov

commit sha 7530fa9914b7f25d002898698fba8a23ac7ff09c

Update bug_report_local_install.md

view details

push time in 10 days

push eventantonbabenko/pre-commit-terraform

Maksym Vlasov

commit sha 62d2180a1535accfb28303a7c1fc1dcb336a764b

Update bug_report_docker.md

view details

push time in 10 days

PR opened antonbabenko/pre-commit-terraform

Update feature_request.md

<!-- Thank you for helping to improve pre-commit-terraform! -->

Put an x into the box if that apply:

  • [ ] This PR introduces breaking change.
  • [x] This PR fixes a bug.
  • [ ] This PR adds new functionality.
  • [ ] This PR enhances existing functionality.

Description of your changes

Fix label auto-adding after label rename <!-- Briefly describe what this pull request does. Be sure to direct your reviewers' attention to anything that needs special consideration.

We love pull requests that resolve an open pre-commit-terraform issue. If yours does, you can uncomment the below line to indicate which issue your PR fixes, for example "Fixes #123": -->

<!-- Fixes # -->

How has this code been tested

<!-- Before reviewers can be confident in the correctness of this pull request, it needs to tested and shown to be correct. Briefly describe the testing that has already been done or which is planned for this change. --> Create new PR&issues

+1 -1

0 comment

1 changed file

pr created time in 10 days

create barnchantonbabenko/pre-commit-terraform

branch : fix/labels

created branch time in 10 days

push eventantonbabenko/pre-commit-terraform

Maksym Vlasov

commit sha ee145f1d78cf6c9c50fe11e9b7f08e4bfab66848

Update label names

view details

push time in 11 days

startedim2nguyen/rover

started time in 11 days

startedTierMobility/boring-registry

started time in 11 days

starteddiggerhq/infragenie

started time in 11 days

delete branch antonbabenko/pre-commit-terraform

delete branch : docs/improve

delete time in 11 days

create barnchantonbabenko/pre-commit-terraform

branch : docs/improve

created branch time in 11 days

PR opened antonbabenko/pre-commit-terraform

Reviewers
fix: GH stale action - remove unsupported setting kind/bug/have_workaround estimate/1h

<!-- Thank you for helping to improve pre-commit-terraform! -->

Put an x into the box if that apply:

  • [ ] This PR introduces breaking change.
  • [ ] This PR fixes a bug.
  • [ ] This PR adds new functionality.
  • [ ] This PR enhances existing functionality.

Description of your changes

<!-- Briefly describe what this pull request does. Be sure to direct your reviewers' attention to anything that needs special consideration.

We love pull requests that resolve an open pre-commit-terraform issue. If yours does, you can uncomment the below line to indicate which issue your PR fixes, for example "Fixes #123": -->

<!-- Fixes # -->

Fix Warning founded afrer GH action run by removing unsupported parameter.

How has this code been tested

<!-- Before reviewers can be confident in the correctness of this pull request, it needs to tested and shown to be correct. Briefly describe the testing that has already been done or which is planned for this change. -->

Check next run after change merge.

Also, this action have dry-run options as it described in action readme, but for this little fix it will be overkill.

workaround

It just warning, so workaraund is do nothing

+0 -2

0 comment

1 changed file

pr created time in 11 days

create barnchantonbabenko/pre-commit-terraform

branch : fix/remove_warning

created branch time in 11 days

startedscraly/terraform-cheat-sheet

started time in 11 days

startedaws-samples/aws-eks-accelerator-for-terraform

started time in 11 days

startedtilfinltd/aws-extend-switch-roles

started time in 14 days