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

explain naming #7

Merged
merged 1 commit into from
Aug 21, 2019
Merged

explain naming #7

merged 1 commit into from
Aug 21, 2019

Conversation

gajowi
Copy link
Collaborator

@gajowi gajowi commented Aug 21, 2019

Tentative first PR from my fork. Since the change is so trivial and intended for upstream, I've just worked on the fork's master - and not varied the CI config to test the fork. Feel free to tell me if this is OK or not. I'm not sure what is best practice on this point

@paulfantom paulfantom merged commit 6985b0a into cloudalchemy:master Aug 21, 2019
@paulfantom
Copy link
Member

GitHub doesn't recommend working on base branch (in this case master), but that's mostly to allow easier synchronization with a repo from which fork originates. From my side there is no restrictions on the matter of which branch you are using on your fork :)

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