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

Discuss: Maintainability of mansion #42

Open
dennislwm opened this issue May 4, 2021 · 22 comments
Open

Discuss: Maintainability of mansion #42

dennislwm opened this issue May 4, 2021 · 22 comments
Labels
discussion Discussion

Comments

@dennislwm
Copy link
Collaborator

Problem: As more links are added to this repo, the README.md file will be longer.

Solution: Follow the structure of Stack on a budget

README.md is basically a TOC (Table of Contents) and each category has a Markdown file under pages folder.

There will also be fewer merge conflicts if more people contribute to different categories.

@huskypower
Copy link
Collaborator

@ytbryan @hadibolehjadi @ylcheryl @shankjs @ReubenChan @dennislwm Hi Lady and Gentlemen, we have an issue here, may we have your thoughts and directions?

@hadibolehjadi
Copy link
Collaborator

I agree with Dennis, it will definitely allow the front page of mansion to be cleaner. So it'll mean that each section has its own md. We do need to discuss how we want the structure of the repo to evolve from here, like do we want python.md to be under the python folder? Or under a directory known as pages?

What do you guys think?

@huskypower
Copy link
Collaborator

tiddy thing up is the future, moving forward the Readme get longer and losses its appeal and purpose, I had a look at Stack on a budget, and the ideas of documenting is simple and sweet , like hadi mentioned we need to work on the structure .

@dennislwm dennislwm added the discussion Discussion label May 4, 2021
@shankjs
Copy link
Collaborator

shankjs commented May 4, 2021

Thanks Dennis, Stack on a budget has a nice streamline repo. They have a nice contributing.md that encourages contribution and how to go about it for quicker reviews and merging.. I agree that we can apply it to the mansion. We can have a think on how the structure can be

@huskypower
Copy link
Collaborator

Calling @ReubenChan and our lady @ylcheryl your thoughts please for us to move on as we are mentioned to proceed with #42 first

@ReubenChan
Copy link
Contributor

Hi Team,
Sorry for the delay.. agreed on the proper structure of our repo. I have seen the stack on a budget. We should emulate the clean structure and beautify our own ideas ... I believe Cheryl will agree as well... Let's move on

@ylcheryl
Copy link

ylcheryl commented May 5, 2021 via email

@ytbryan
Copy link
Owner

ytbryan commented May 5, 2021

Let's do this structure! Stack on a budget has a neat structure for its markdown too. The contributors only need to submit a link, pro, cons. Check out their markdown.

Everyone needs to learn how to merge conflict.

Thanks for bringing this up @dennislwm

@dennislwm
Copy link
Collaborator Author

dennislwm commented May 8, 2021

Hi @shankjs @huskypower @hadibolehjadi @ylcheryl @ReubenChan

Followup on @ytbryan suggestion for a value proposition.

For each resource, I think it should have the following:

Resource Title

  • Content type: [video, blog, podcast, git repo etc]
  • Short description: [short description]
  • Pricing: [free, paid tiers]
  • Limitations: [limitations of free]

This is just a quick start, any comments are welcomed.

@ytbryan
Copy link
Owner

ytbryan commented May 8, 2021

Wow. thanks for this direction @dennislwm brilliant!

Suggestion! We should add some constraint to this to make it succeed faster. For now, we should remove short description. It can come in later (beyond batch 1). Stack on a budget also does not have description.

Welcome the content type field.

Other consideration, will we need more directories instead of one page directory? Or it can come in later?

@shankjs
Copy link
Collaborator

shankjs commented May 9, 2021

Thanks @dennislwm! The structure of content type is definitely a good move forward. There definitely are various forms of contribution. Helps with the classification of content.

Here's a visual representation of the structure. Feel free to use this to tinker on the structure all.

pages.md
|---- azure.md
|---- devops.md
|---- linux.md
|---- python.md
|---- docker.md
|---- github.md

Table of Contents

azure.md

Azure

Documentation

Content: Documentation | Ref: Microsoft

Exam Skills Measurement

Content: Website | Ref: Microsoft

Command-Line Installation

Content: Website | Ref: Microsoft

devops.md

DevOps

DevOpsGroup

Content: Blog

@ytbryan
Copy link
Owner

ytbryan commented May 12, 2021

I think we should retain our current directory. Note: something & something_else are just names. You decide.

azure/
|------ something.md
|------ something-else.md
devops/
|------ something.md
|------ something-else.md
python/
|------ something.md

instead of

pages/
|---- azure.md
|---- devops.md
|---- linux.md
|---- python.md
|---- docker.md
|---- github.md

directory name should be lower case. markdown name should be lowercase and separated with hyphen. Names must be meaningful and accurate.

@huskypower
Copy link
Collaborator

@ytbryan you mean the Folder structure as pre below?

azure/
|------ something.md
|------ something-else.md
devops/
|------ something.md
|------ something-else.md
python/
|------ something.md

the there will be one, two or more .md files? and theses .md files are links to the topics? or are we able to submit files actual files in the folders sure as azure/devops/etc.

@huskypower
Copy link
Collaborator

@ytbryan what I meant was

for the folder structure above you mentioned

example:

python/
|------something.md
|------somethinelse.md

that something.md aand somethingelse.md must it be markdown file or can they be actual files such as challenges script and others video?

@ytbryan
Copy link
Owner

ytbryan commented May 15, 2021

Oh! I think it should be only markdown files.

@huskypower
Copy link
Collaborator

@ytbryan we are working on the readme.md file using this structure below?

azure/
|------ something.md
|------ something-else.md
devops/
|------ something.md
|------ something-else.md
python/
|------ something.md

@ytbryan
Copy link
Owner

ytbryan commented May 15, 2021

azure/
|------ something.md
|------ something-else.md
devops/
|------ something.md
|------ something-else.md
python/
|------ something.md
readme.md

There should be a readme.md file at the highest level. My point is that we already have such directory structure.

@huskypower
Copy link
Collaborator

@ytbryan Agree the readme.md is the TOC (table of content) which looks slick and neat at the first page and we have the content , links in the folders as per the TOC.
Content type: [video, blog, podcast, git repo etc]
In the folders eg,

azure/
|-----content type (video)
|-----content type (blog)
|-----content type(git repo)
|-----content type(others to be addedin)

We can either prelist the popular content type in the folder, ready for contributors to add in and or when contributor have a new contact type not in the folder to send a issue request for collaborators to add in and review.

@ReubenChan
Copy link
Contributor

ReubenChan commented May 15, 2021

I'm onboard with the new table of content(TOC). It's like our signature design for better visuals.

Or would you like to add a collapsible header for the title for example:

Azure

    @ytbryan
    Copy link
    Owner

    ytbryan commented May 15, 2021

    any example repo with the usage? @ReubenChan

    @ReubenChan
    Copy link
    Contributor

    I think better not to apply the collapsible title header because of more technical complication when appending with sub titles. @huskypower, @shankjs, and @ytbryan the TOC example looks more structured, neat & tidy

    @shankjs
    Copy link
    Collaborator

    shankjs commented May 16, 2021

    The TOC is based on a different structure than we currently have. The example was to see how different it would be from our current structure. I have to agree it looks more structured but that would mean changing the current structure. Adding markdowns to the current structure would be easier for all.

    azure/
    |------ something.md
    |------ something-else.md
    devops/
    |------ something.md
    |------ something-else.md
    python/
    |------ something.md
    readme.md

    suggestion for various content type(s), they can be linked within the markdown
    i.e.
    -[Youtube - Installing ubuntu 18.04 on virtual box ](URL)

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    discussion Discussion
    Projects
    None yet
    Development

    No branches or pull requests

    7 participants