Skip to content

remove: anko - #181

Open
dhruv1294 wants to merge 2 commits into
delta:dalal-street-21from
dhruv1294:replaceAnko
Open

remove: anko#181
dhruv1294 wants to merge 2 commits into
delta:dalal-street-21from
dhruv1294:replaceAnko

Conversation

@dhruv1294

Copy link
Copy Markdown
Member

No description provided.

@dhruv1294
dhruv1294 requested a review from ranaaditya January 24, 2021 09:39
import org.pragyan.dalal18.utils.Constants
import org.pragyan.dalal18.utils.MiscellaneousUtils
import org.pragyan.dalal18.utils.viewLifecycle
import org.pragyan.dalal18.utils.*

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove the wild card imports.

Suggested change
import org.pragyan.dalal18.utils.*
import org.pragyan.dalal18.utils.ConnectionUtils
import org.pragyan.dalal18.utils.Constants
import org.pragyan.dalal18.utils.MiscellaneousUtils
import org.pragyan.dalal18.utils.viewLifecycle

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

File > Settings > Editor > Java > General > Class count to use import with '*'

import org.jetbrains.anko.longToast
import org.jetbrains.anko.toast
import org.jetbrains.anko.uiThread
import kotlinx.coroutines.*

@avikumar15 avikumar15 Jan 24, 2021

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here

Suggested change
import kotlinx.coroutines.*
import kotlinx.coroutines.Dispatchers
import kotlinx.coroutines.launch
import kotlinx.coroutines.withContext

import org.pragyan.dalal18.databinding.FragmentMyOrdersBinding
import org.pragyan.dalal18.utils.ConnectionUtils
import org.pragyan.dalal18.utils.Constants
import org.pragyan.dalal18.utils.*

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as above

@ranaaditya ranaaditya linked an issue Feb 3, 2021 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replacing Anko completely from the codebase

3 participants