Skip to content

Add docs on DaRUS automation#361

Merged
uekerman merged 5 commits intomasterfrom
darus-automation
Apr 19, 2024
Merged

Add docs on DaRUS automation#361
uekerman merged 5 commits intomasterfrom
darus-automation

Conversation

@uekerman
Copy link
Member

As we have no good place yet where to upload the scripts and the scripts need manual editing anyway, I thought adding them in plain text here could be a good compromise.

The actual component versions still need final updates.

@MakisH No need to test anything. A quick check should be enough. I will anyway "test this" with the real data.

Thanks @JR-1991 for the help setting this up 🙏 Should be added as a co-author here when merging.

@uekerman uekerman requested a review from MakisH March 26, 2024 09:00
@uekerman uekerman self-assigned this Mar 26, 2024
Copy link
Member

@MakisH MakisH left a comment

Choose a reason for hiding this comment

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

Basic sanity check makes sense. I will approve when the final versions are there.

Copy link
Member

@MakisH MakisH left a comment

Choose a reason for hiding this comment

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

Actually, this is ready, as we don't need to update the versions here. I created a separate PR for that: #380

@uekerman uekerman marked this pull request as ready for review April 17, 2024 06:50
@uekerman
Copy link
Member Author

@MakisH In #380 will you then also update the versions on this page? (currently you don't)

@MakisH
Copy link
Member

MakisH commented Apr 17, 2024

No: the versions will then always need to be updated, and they will be duplicated. I find TODOs better for this template.

@uekerman
Copy link
Member Author

we could also leave the versions of the current distribution and document that this is an example and needs to be updated before using.
if not, then we should probably switch to TODOs here everywhere

@uekerman
Copy link
Member Author

I now added the versions and a bit of documentation. Simply saves me one TODO when I need to actually apply the script.

@uekerman uekerman merged commit d926e2c into master Apr 19, 2024
@uekerman uekerman deleted the darus-automation branch April 19, 2024 13:32
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