Skip to content
This repository has been archived by the owner on Apr 13, 2020. It is now read-only.

fix typo #65

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix typo #65

wants to merge 1 commit into from

Conversation

yelizariev
Copy link
Contributor

No description provided.

@yajo
Copy link
Contributor

yajo commented Jan 7, 2020

Is this "typo" on purpose, @pedrobaeza?

@pedrobaeza
Copy link
Member

This commit explains better why oca is there: c2252fe

Maybe we should homogeneize the sample with oca word in current remote?

@yajo
Copy link
Contributor

yajo commented Jan 13, 2020

This is why I prefer docs over samples... 🤦‍♂️

We have a builtin snippet included for vscode, where you type repo and it fills as needed:

Peek 13-01-2020 10-13

That, along with the link supplied at the top of the file, should be enough boilerplate for this file IMHO.

If we have to homogenize, I'd use origin, as it's clearer to understand.

@pedrobaeza
Copy link
Member

The problem is that the Odoo repo is special as we set several sources, and we choose by default ocb, not "origin" Odoo.

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