Skip to content

Create README files at domain and root level also#41

Merged
rajatgoyal715 merged 5 commits intomasterfrom
feature/root-level-readme
Oct 29, 2019
Merged

Create README files at domain and root level also#41
rajatgoyal715 merged 5 commits intomasterfrom
feature/root-level-readme

Conversation

@rajatgoyal715
Copy link
Member

Closes #39

Changelogs

  • Create README files at the domain level and root level
  • Refactoring current code to make it more readable

@rajatgoyal715 rajatgoyal715 added the enhancement New feature or request label Oct 25, 2019
@rajatgoyal715 rajatgoyal715 added this to the 1.3.0 milestone Oct 25, 2019
@rajatgoyal715 rajatgoyal715 self-assigned this Oct 25, 2019
@rajat19
Copy link
Member

rajat19 commented Oct 26, 2019

Some checks failed
Please look into it

@rajatgoyal715
Copy link
Member Author

Some checks failed
Please look into it

Looking into these!

hsc/crawler.py Outdated

if self.make_language_folder:
folder_name = os.path.join(folder_name, language)
# if self.make_language_folder:
Copy link
Member

Choose a reason for hiding this comment

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

should specify reason for commenting this

Copy link
Member Author

Choose a reason for hiding this comment

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

We weren't using this anyways. Can you please help me out here if we are going to use this in the future?

Copy link
Member

Choose a reason for hiding this comment

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

We haven't exposed this functionality yet. It was to be exposed in #24 as mentioned here. But we forgot to add it. >.<

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed this in 14f41fa

@rajatgoyal715 rajatgoyal715 merged commit cc7ded3 into master Oct 29, 2019
@delete-merged-branch delete-merged-branch bot deleted the feature/root-level-readme branch October 29, 2019 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create root level and domain level README files

3 participants