Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

florianow
Copy link
Collaborator

No description provided.

feat: GCP budget alert building block

ci: adding changes to the validate process

feat: GCP budget alert building block
feat: GCP budget alert building block

ci: adding changes to the validate process

feat: GCP budget alert building block

chore: adding tf stuff to git ignore

feat: GCP budget alert building block

ci: adding changes to the validate process

feat: GCP budget alert building block

chore: adding tf stuff to git ignore

chore: adding tf stuff to git ignore
ci: test script

ci: test script

ci: test script

ci: test script

ci: test script

ci: test script

ci: test script
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-44.d1o16zfeoh2slu.amplifyapp.com

ci: adding everything to pre-commit

ci: adding everything to pre-commit

ci: adding everything to pre-commit
@florianow florianow force-pushed the feature/validate branch 2 times, most recently from d49d936 to 0bd5449 Compare June 20, 2025 12:59
chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules
@florianow florianow force-pushed the feature/validate branch 2 times, most recently from 3129081 to e5a5616 Compare June 20, 2025 14:32
chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding "how to contribute"

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding "how to contribute"

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding "how to contribute"

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding "how to contribute"

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding "how to contribute"

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding "how to contribute"

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding "how to contribute"

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules

chore: adding files to modules
uses: actions/checkout@v4

- uses: meshcloud/setup-collie@main
- name: Set up meshcloud collie CLI
Copy link
Member

Choose a reason for hiding this comment

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

d: no longer needed?

#!/bin/bash
set -e

warnings=()
Copy link
Member

Choose a reason for hiding this comment

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

d: i appreciate that AI generated script, but can't we use sth builtin for tflint maybe?

(don't spend a ton of time reseaching this, the script is fine)

if ! grep -q "supportedPlatforms:" "$readme_path"; then
errors+=("Missing 'supportedPlatforms' in README.md at $readme_path")

# 3. Extract YAML block
Copy link
Member

Choose a reason for hiding this comment

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

I#d rather appreciate we did a proper jq parsing of this thing? iirc there's also yq (jq but for yaml) that we could just get via nix?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

true, that exists :) I will use it


for png_file in $(find $modules_glob -name '*.png'); do
# Check PNG files only directly inside each buildingblock
for png_file in $(find $modules_glob -maxdepth 1 -name '*.png'); do
Copy link
Member

Choose a reason for hiding this comment

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

you know what I'd also love to have: automatically optimizing the png files (meshStack needs them small) with sth like optipng and making sure they are as small as possible (lossless ofc)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

good idea

@@ -0,0 +1,7 @@
provider "azurerm" {
Copy link
Member

Choose a reason for hiding this comment

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

tbh I'm probably against having empty default provider.tfs as that makes it very difficult to inject additional provider config via terragrunt? If I'm overriding the providers then I need to also user provider.tf as a name to make 100% sure I override the right one.

For BBs our standard pattern is "just use env vars supported by the providers for configuration" and that seems to be also the standard pattern for reusable terraform modules

we should 100% have versions.tf files

and for BBs maybe even lockfiles (for all architectures... which is a PITA)

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