Sitelet https://github.com/TheRenegadeCoder/sample-programs-website/pull/334
Skip to content

Added Duplicate Character Counter in Every Language Article - #334

Merged
jrg94 merged 12 commits into
TheRenegadeCoder:masterfrom
ElizabethGraham:duplicate-char-counter
Oct 11, 2020
Merged

jrg94 merged 12 commits into
TheRenegadeCoder:masterfrom
ElizabethGraham:duplicate-char-counter

Conversation

@ElizabethGraham

Copy link
Copy Markdown
Contributor

If any changes are needed, let me know!

@jrg94 jrg94 left a comment

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.

Looks good! A couple quick changes, and I think we'll be good to go.

Comment thread projects/duplicate-char-counter/_posts/2018-12-15-python.md Outdated
Comment thread projects/duplicate-char-counter/_posts/2020-10-04-python.md Outdated
Comment thread projects/duplicate-char-counter/index.md Outdated
Comment thread projects/duplicate-char-counter/index.md Outdated
Comment thread projects/duplicate-char-counter/index.md Outdated
@jrg94 jrg94 self-assigned this Oct 4, 2020
@jrg94 jrg94 added the project Project documentation label Oct 4, 2020
@ElizabethGraham

Copy link
Copy Markdown
Contributor Author

Think I got it all :)

@jrg94 jrg94 changed the title Added duplicate character counter documentation Added Duplicate Character Counter Documentation Oct 5, 2020

@jrg94 jrg94 left a comment

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.

This is awesome! I have a quick question about the output. Also, did you mean to delete the article? It looked like there were two of them, but one of them was fine.

Comment thread projects/duplicate-char-counter/index.md Outdated
@jrg94 jrg94 changed the title Added Duplicate Character Counter Documentation Added Duplicate Character Counter in Every Language Article Oct 5, 2020

@jrg94 jrg94 left a comment

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.

Looks great! I just have one last question about the format of the output, so we can begin writing up the testing.

Comment thread projects/duplicate-char-counter/index.md Outdated
Co-authored-by: Jeremy Grifski <jeremy.grifski@gmail.com>
@ElizabethGraham

Copy link
Copy Markdown
Contributor Author

Looks great! I just have one last question about the format of the output, so we can begin writing up the testing.

Yeah, it's meant to be separate lines, but the formatting above broke the table for me.

@jrg94

jrg94 commented Oct 6, 2020 •

Copy link
Copy Markdown
Member

You're 100% right! Here's how we did it for fibonacci: https://raw.githubusercontent.com/TheRenegadeCoder/sample-programs-website/master/projects/fibonacci/index.md

Maybe, we can use <br /> here.

@jrg94 jrg94 left a comment

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.

We're in great shape! Did you ever add yourself to the authors file? https://github.com/TheRenegadeCoder/sample-programs-website/blob/master/_data/authors.yml

@jrg94

jrg94 commented Oct 7, 2020

Copy link
Copy Markdown
Member

Woah that was weird timing!

@ElizabethGraham

Copy link
Copy Markdown
Contributor Author

Ok awesome! I think I got it all sorted

@ElizabethGraham

Copy link
Copy Markdown
Contributor Author

lmao that was really weird

@ElizabethGraham

Copy link
Copy Markdown
Contributor Author

I'll go ahead and add myself to the authors rn haha

@ElizabethGraham

Copy link
Copy Markdown
Contributor Author

Thanks a bunch for being patient and helping me so much! I know a little markdown now as a result lol

@jrg94

jrg94 commented Oct 7, 2020

Copy link
Copy Markdown
Member

Oh no problem! I was actually a bit more worried that this was too complicated. Let me know if there's anything we can do to simplify this process more.

@ElizabethGraham

Copy link
Copy Markdown
Contributor Author

Is everything set for a merge? :)

@jrg94

jrg94 commented Oct 8, 2020 •

Copy link
Copy Markdown
Member

We're missing your entry in the author.yml file, I think. @GoodbyeBlues

@ElizabethGraham

Copy link
Copy Markdown
Contributor Author

Just added it. My bad, I tried making a pull request for that a while ago but I guess it didn't go through!

@jrg94 jrg94 left a comment

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.

👍

@jrg94
jrg94 merged commit ac27cc3 into TheRenegadeCoder:master Oct 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

project Project documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants