Skip to content
This repository has been archived by the owner on Nov 20, 2024. It is now read-only.

Feature: improved initialisation commands #13

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

marblegravy
Copy link

@marblegravy marblegravy commented Sep 7, 2019

Following on from the troubleshooting in #12 I've modified the ahoy files to include a few extra options to stop anyone else running in to similar issues.

My scenario is uncommon because I have multiple versions of govCMS scaffolds for different purposes, but what I didn't understand was that my other projects were 'leaking' their modified containers in to this one.

It's possible that others could find themselves in a similar boat in future.

To mitigate this, I've proposed that the readme uses a different command ahoy init when you first set up the project that will force it to use the exact containers specified and not cached versions that might already exist on the users machine.

I've also created an additional command ahoy rebuild that does the same thing but with the idea that if someone has been working on a project for a long time and updates have been made upstream, that they may want to effective force refresh and get the latest version of everything back to a known good state.

Finally, I've added in a link to the Lagoon troubleshooting section (from the old lagoon docs as this section hasn't been moved to the new version yet that I could see) at the bottom of the readme.

Hopefully these small mods should mean all future users can reliably generate the scaffold exactly as intended regardless of what else they may have installed previously.

@simesy
Copy link
Contributor

simesy commented Sep 8, 2019

My view expressed elsewhere is that the ahoy file needs a massive shakedown.

@marblegravy
Copy link
Author

@simesy - shakedown how? To simplify it, just re-write it, add more functions or a combo of the two?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants