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

Rename Fake to Demo #1291

Merged
merged 8 commits into from
Mar 21, 2024
Merged

Rename Fake to Demo #1291

merged 8 commits into from
Mar 21, 2024

Conversation

Jaehwa-Noh
Copy link
Contributor

@Jaehwa-Noh Jaehwa-Noh commented Mar 16, 2024

What I have done and why
Rename Fake to Demo.
When name is Fake, it is make analysis hard. The problem is naming.
Actually it is not fake because it is for demo to build on not product code.
The only NiaNetworkDataSource reference is FakeNiaNetworkDataSource in non-production build.
So, I think it is better to rename Fake to Demo.

Change-Id: I5b1d7963e623832a766900aa74ef1085aac0cacc
Change-Id: Icd7005ec38495a1a4bc6f1594f70a2c1228538ab
Change-Id: I48bb3c26ef02d6302cb7caabcaa17d5b4659031a
This reverts commit a5be299.
Change-Id: I3b1e19dedd238ec96bd8debee5fcbcc91bc25094
Change-Id: Ife321dd95238227bfdfc1f13f6832472bd699b6f
Change-Id: I025fc00aab56c998c64bd88ad337cba8061d507c
@dturner
Copy link
Collaborator

dturner commented Mar 21, 2024

You make an excellent point. I agree, "Demo" is more accurate than "Fake". Thanks.

@dturner dturner merged commit ecdfdef into android:main Mar 21, 2024
4 checks passed
Jaehwa-Noh added a commit to Jaehwa-Noh/Jaehwa-Noh that referenced this pull request Mar 21, 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

Successfully merging this pull request may close these issues.

2 participants