Skip to content

Feature/validate #44

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Feature/validate #44

wants to merge 7 commits into from

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: 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

check_readme_format "$readme_file"
done

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