varia
/
etherpump
12
0
Fork 0
Code Issues 15 Pull Requests Releases Activity
Labels Milestones
New Issue

To keep the API interface simple: what about only using the option "opt_in=True" #13

Open
opened 3 years ago by mb · 2 comments
mb commented 3 years ago

(API thoughts that boiled up in a conversation @ pub.club)

(API thoughts that boiled up in a conversation @ pub.club)
mb commented 3 years ago
Poster

Another suggestion is to also add the folder="FOLDERNAME".

Another suggestion is to also add the folder="FOLDERNAME".
decentral1se commented 3 years ago
Owner

One idea is take all command line arguments as they stand on the command line and thread them through via the magic word API interface. So if you use --publish-opt-in on the CLI you pass publish_opt_in=True in the function definition.

Then the other idea was that the command line interface is confusing so we should make it easier to use when writing th script. But of course, we're just making new conventions and running into new confusions with that :)

Anyway, just writing before I forget.

One idea is take all command line arguments as they stand on the command line and thread them through via the magic word API interface. So if you use `--publish-opt-in` on the CLI you pass `publish_opt_in=True` in the function definition. Then the other idea was that the command line interface is confusing so we should make it easier to use when writing th script. But of course, we're just making new conventions and running into new confusions with that :) Anyway, just writing before I forget.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
add-new-addtext-command
add-quote-import
main
publish-vs-nopublish
Labels
Apply labels
Clear labels
No items
No Label
Milestone
Set milestone
Clear milestone
No items
No Milestone
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch '%!s(MISSING)'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes