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

Clarify users that a create_index field is needed in the Settings.toml file #96

Merged
merged 3 commits into from
Feb 6, 2024

Conversation

zenodeapp
Copy link
Contributor

This is a clarification on how to fix the issue explained in #95.

@CLAassistant
Copy link

CLAassistant commented Feb 5, 2024

CLA assistant check
All committers have signed the CLA.

@zenodeapp
Copy link
Contributor Author

zenodeapp commented Feb 5, 2024

If default should be false tell me!

@rllola
Copy link
Contributor

rllola commented Feb 5, 2024

@zenodeapp Can you accept the CLA and I can merge this

@zenodeapp
Copy link
Contributor Author

@zenodeapp Can you accept the CLA and I can merge this

Done! Was in a phone call, forgive me.

@zenodeapp
Copy link
Contributor Author

Wasn't aware that this PR was still open. I accidentally merged with recent changes again! Sorry!

@rllola
Copy link
Contributor

rllola commented Feb 6, 2024

No problem @zenodeapp. We are all good now. Merging it.
Thanks for your contribution.

@rllola rllola merged commit 6abb737 into Zondax:main Feb 6, 2024
5 of 6 checks passed
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.

3 participants