@starbelly, I'm going to ask for your review here. There's nothing special in the pull request but some extra eyes are always nice. You could probably also run this over that project you have with multiple rebar3 versions and whatnot, even using Windows, maybe (?)

I learned, yesterday, that .addPath existed. It's a nice replacement for this.

This is actually just moving code around, to revert a previous decision that created more confusion that expected.

I got into trouble with powershell and paths. This removed my obstacles.

Yeah, so the new approach is to find erl.exe and make the container folder part of the path. 🎉 , more Windows versions supported.

Removed what was unnecessary, even if old code. It makes for easier maintenance.

I got into trouble with powershell and paths. This removed my obstacles.

I split the CI files into three different files. Rationale: easier to see where an error comes from, easier to have more inspectable job names.

Let me add some self-review comments, for clarity. FWIW, this pull request closes two outstanding issues: #50 and #51, but created recently.

Moving away from draft.

This is probably not very important. The day support is removed the action can be consumed with that version either, so we can leave this here until then and then simply remove whatever code/tests are there for ubuntu-2016.

1. support more OTP+Windows versions; try to bridge the gap with gleam-lang/setup-erlang, that seems to support pre-21 versions

Opening as draft.

As-is, stuff appears to be working (and you're Ok to review, since I don't expect this to change much) but a couple of questions reside. I'll comment on the code later.

Closes #12. Closes #44.

Edit: one big question for me is "Should we open an exception to the Windows installation script in order to support an even further past"?

At the moment it's Linux only.

The error you get is not from the fact the file is not available, which it should be, but further down the execution, by stating the version is not found in the listing.

When trying to download whatever file or fetching whatever external dependency we shouldn't hide errors behind default return values, meaning we should crash whenever https://github.com/erlef/setup-beam/blob/main/src/setup-beam.js#L272 is invoked.

I'm merging this to start working on #51. This is usable as-is, in any case. Feel free to comment further here or in #51 and I'll do my best to approach your suggestions.

@ripienaar Feeds and Chat are free with the Maker Account (https://getstream.io/maker-account/) which is accessible via both pricing pages.

The new plan provides qualifying teams with full and free access to Stream’s Chat API (2k MAU) and Activity Feed API (9 million feed updates/month).

Full details here: https://getstream.io/blog/maker-account/

There is a discrepancy in the example that confuses me:

infrastructure/sources/podinfo.yaml does not specify a namespace, so the HelmRepository is placed in the default namespace.

apps/base/podinfo/release.yaml specifies as its sourceRef a HelmRepository named podinfo but in the flux-system namespace.

Should these two not refer to the same namespace?

Thanks. I missed that subtlety when simplifying.

<!--- Please keep this note for the community --->

Community Note

• Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
• Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for pull request followers and do not help prioritize the request

<!--- Thank you for keeping this note for the community --->

<!--- If your PR fully resolves and should automatically close the linked issue, use Closes. Otherwise, use Relates ---> Closes #94

Release note for CHANGELOG: <!-- If change is not user facing, just write "NONE" in the release-note block below. -->

Currently the operator defaults to naming the workspace in a predetermined format "[namespace]-[name]." This PR adds a new optional flag in the k8s object to omit the namespace prefix from the workspace name. This enables complete control over the workspace name.

In k8s custom resource workspace <!-- Replace with relevant outputs or interfaces -->

---
apiVersion: app.terraform.io/v1alpha1
kind: Workspace
metadata:
    name: greetings
spec:
    secretsMountPath: "/tmp/secrets"
    module:
        source: "git::https://github.com/koikonom/tfc-operator-test.git"
    omitNamespacePrefix: true
...

The name of this workspace is "greetings."

Errors were reported while checking the availability of links.

📝 Summary
---------------------
🔍 Total..........600
✅ Successful.....595
⏳ Timeouts.........0
🔀 Redirected.......0
👻 Excluded.........4
🚫 Errors...........1

Errors in README.md
✗ https://languagetool.org/ (HTTP status client error (403 Forbidden) for url (https://languagetool.org/))

Full Github Actions output

see title