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

Request: Rename only_tables to only in config yaml 🙏 #294

Open
gsweats opened this issue Feb 4, 2024 · 0 comments
Open

Request: Rename only_tables to only in config yaml 🙏 #294

gsweats opened this issue Feb 4, 2024 · 0 comments

Comments

@gsweats
Copy link

gsweats commented Feb 4, 2024

Hello! So excited to have stumbled across Replibyte today! As a newbie, I was trying to export only a certain table. I didn't see anything in the beautiful doc site (nice work!) but I searched the source and saw only_tables. Assuming that was just a variable name (I don't rust yet) I mistakingly used only: in my conf.yaml, emulating the skip: option, which is documented.

Long story short, please can we rename only_tables in the YAML schema to only and have some kind of legacy alias to continue supporting only_tables (perhaps with a console warning?) but officialy promote only: in the docs?

Merely a new user here, I might be missing something. Next Issue will be to add more to the docs, like official API documentation (must be some kind of rustdoc type thing).

Thanks, best!

@gsweats gsweats changed the title Rename only_tables to only in config yaml Rename only_tables to only in config yaml Feb 4, 2024
@gsweats gsweats changed the title Rename only_tables to only in config yaml Recommendation: Rename only_tables to only in config yaml Feb 4, 2024
@gsweats gsweats changed the title Recommendation: Rename only_tables to only in config yaml Request: Rename only_tables to only in config yaml 🙏 Feb 4, 2024
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

No branches or pull requests

1 participant