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

FIxed README and Variable descriptions #38

Closed
wants to merge 84 commits into from
Closed

Conversation

herms14
Copy link
Collaborator

@herms14 herms14 commented Feb 14, 2024

Updated README file to make location a required variable
Updated variable descriptions based on AVM format

* Create CODEOWNERS

* Update CODEOWNERS

* Update CODEOWNERS

* Update CODEOWNERS
* Update avmrequiredfiles.yml

* Update avmrequiredfiles.yml

* Update CODEOWNERS
* feat: structure

* add tests dir

* test add .go file
* Update CODEOWNERS

* Update CODEOWNERS
…zure#17)

Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v3...v4)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Update main.telemetry.tf

renamed telemetry resource name to be consistent with local variables and removed location argument as it is not needed

* updated readme
jtracey93 and others added 11 commits January 23, 2024 21:35
* Update _header.md

* Update _header.md

Co-authored-by: Máté Barabás <barabasmate@gmail.com>

* Update _header.md

Co-authored-by: Máté Barabás <barabasmate@gmail.com>

---------

Co-authored-by: Máté Barabás <barabasmate@gmail.com>
* feat: add docker pull to avm scripts to ensure latest

* feat: update avm script to always pull latest image
* fix: docker pull
* Update readme

* docs: update header

---------

Co-authored-by: Matt White <16320656+matt-FFFFFF@users.noreply.github.com>
Azure#83)

* Ignore `.DS_Store` and `*.md.tmp` files

* Adjusted TODO references to keep TF linter green for fresh modules

* adjusted
@matt-FFFFFF
Copy link
Member

@herms14 lease can you pull in the changes from main? This branch is using the old actions and they are all failing.

@matt-FFFFFF
Copy link
Member

You should rebase your branch with Azure/main

2 file(s) have code issues.

File Issues
.github/policies/avmrequiredfiles.yml While scanning a multiline plain scalar, found invalid mapping.
.github/policies/branchprotection.yml While scanning a multiline plain scalar, found invalid mapping.

Total execution time: 0.04 seconds

@herms14 herms14 closed this Mar 8, 2024
@herms14
Copy link
Collaborator Author

herms14 commented Mar 8, 2024

closing PR. Will submit a new one

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.