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

Add self from OUJS Organization to AUTHORS.md #1233

Merged
merged 12 commits into from
Apr 20, 2018
Merged

Add self from OUJS Organization to AUTHORS.md #1233

merged 12 commits into from
Apr 20, 2018

Conversation

Martii
Copy link
Contributor

@Martii Martii commented Apr 19, 2018

Technically this makes me a Contributor according to GH ;) LOL j/k I do what I can when I can. :)

Marked version:

0.3.19

Markdown flavor: Markdown.pl|CommonMark|GitHub Flavored Markdown|n/a
GitHub Flavored Markdown (Perhaps CommonMark down the line)

Description

Expectation

GitHub Flavoured Markdown of course.

Result

Continued, awesome, markdown for > ~9437 users ... denoted.

What was attempted

A subtle, indirect, request for Usership addition in a comment. :) Hopefully I got the badge of honour correct.

Contributor

  • Test(s) exist to ensure functionality and minimize regression (if no tests added, list tests covering this PR); or,
  • no tests required for this PR.
  • If submitting new feature, it has been documented in the appropriate places.

Committer

In most cases, this should be a different person than the contributor.

  • Draft GitHub release notes have been updated.
  • CI is green (no forced merge required).
  • Merge PR

Technically this makes me a Contributor according to GH ;) LOL j/k I do what I can when I can. :)

<!--

        If release PR, add ?template=release.md to the PR url to use the release PR template.

        If badging PR, add ?template=badges.md to the PR url to use the badges PR template.

        Otherwise, you are stating this PR fixes an issue that has been submitted; or,
        describes the issue or proposal under consideration and contains the project-related code to implement.

-->

**Marked version:**

0.3.19

<!-- The NPM version or commit hash having the issue -->

**Markdown flavor:** Markdown.pl|CommonMark|GitHub Flavored Markdown|n/a
GitHub Flavored Markdown *(Perhaps CommonMark down the line)*

## Description

- Fixes the request at #1232 (comment) <!-- (if fixing a known issue; otherwise, describe issue using the following format) -->

<!--
        If no issue exists that you're aware of. The maintainers should be able to figure out if it's a duplicate.
-->

## Expectation

<!--
Describe the output you are expecting from marked
-->

GitHub Flavoured Markdown of course.


## Result

<!--
Describe the output you received from marked
-->

Continued, awesome, markdown for > ~9437 users ... denoted.

## What was attempted

<!--
Describe what code combination got you there
-->

A subtle, indirect, request for Usership addition in a comment. :) Hopefully I got the badge of honour correct.


## Contributor

- [ ] Test(s) exist to ensure functionality and minimize regression (if no tests added, list tests covering this PR); or,
- [ ] no tests required for this PR.
- [ ] If submitting new feature, it has been documented in the appropriate places.

## Committer

In most cases, this should be a different person than the contributor.

- [ ] Draft GitHub release notes have been updated.
- [ ] CI is green (no forced merge required).
- [ ] Merge PR
* Usually this is set to these values but I turned it off for this PR in case it blew up GH md. Had this happen once on another project. So crosses-fingers and things

Ref:
* https://github.com/markedjs/marked/blob/92f0ae232f48440cc1fbdcf91b7e3d87226ae05d/.editorconfig
@Martii
Copy link
Contributor Author

Martii commented Apr 19, 2018

Think it's ready for review. Cheers.

* Wikipedia has their own redirects for browser language used for this link
* Also lets portables redirect to mobile page if available.
@UziTech UziTech requested a review from joshbruce April 20, 2018 00:02
UziTech
UziTech previously approved these changes Apr 20, 2018
* Don't know what I was thinking. :) okay enough O.C. for now.
@Martii
Copy link
Contributor Author

Martii commented Apr 20, 2018

Some interesting features this project utilizes on GH... sorry for the noise. Still getting used to these features. May adopt WIP at some point for our site as that's important.

docs/AUTHORS.md Outdated
|Name |GiHub handle |Decision making |Badges of honor (tag for questions) |
|:--------------|:--------------|:----------------------------------------------------|------------------------------------|
|Marti Martz |@Martii |OpenUserJS.org Co-Owner / Project Active Maintainer |Open source, of course |

Copy link
Member

Choose a reason for hiding this comment

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

@Martii thanks, we hadn't figured out what to do on these tables. I was thinking it would be different than the others.

Maybe:

Individual or organization Website Submitted by
OpenUserJS.org https://openuserjs.org Marti Martz (@Martii) Co-Owner / Project Active Maintainer

If you would like to take the game dynamics (badges) to the OpenUserJS project right on!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That looks really good... plus it seems narrower for those not using the Wide GitHub Usercript by xthexder (I try to take these things into account). May put in the @ symbol to our organization home @OpenUserJS since the linkage is in the website.... the .org is the project.

Will change momentarily.

Copy link
Member

Choose a reason for hiding this comment

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

Oooooh! That @ thing...maybe a different column?? GitHub project - a lot of our users are other open source projects.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

maybe a different column??

Depends on how wide you want it. I use the aforementioned Userscript constantly but not everyone does so pushing the boundaries with more columns will make more word wraps. The reason I put spaces in between the / but didn't seem to do much. GFM can be picky sometimes.

We can try it if you like.

Copy link
Member

Choose a reason for hiding this comment

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

I'm not too concerned about the width and word wraps from a PO perspective but it's a valid point. I also like the idea of linking directly to the project. Maybe something like this (adding another use to demonstrate the added value to other users:

Individual or organization Website GitHub Project Submitted by
OpenUserJS.org https://openuserjs.org Marti Martz (@Martii) Co-Owner / Project Active Maintainer
Node Package Manager https://www.npmjs.com https://github.com/npm/npm --

See how some users may not have the direct one-to-one you have??

Copy link
Member

Choose a reason for hiding this comment

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

Note: The word wraps will potentially look and feel different here in the code review than on display on our site or by GH later.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ughh GH isn't updating your comments real time... totally missed these... will read after phone call...

* No badge column now so remove linkage reference
* Not everyone's project name matches their DN so that's why I put the `@` symbol in for the Organization *(username on GH)*
@joshbruce
Copy link
Member

@Martii: You know what, I think I'm asking you do something that might be easier if I just edited this directly. Would you mind me doing that?

* This is the nomenclature and the compromise that can remove the `@` symbol by linking... similar to mentioning GH teams
* Using relative URLS unless you plan on putting this on a custom domain
* Since no-one elses `@`nick mention is hyperlinked not doing mine for symmetry
* Not everyones project is hosted on GH btw... so `Project` is neutral... typically we use the word `Development`
@Martii
Copy link
Contributor Author

Martii commented Apr 20, 2018

@joshbruce
Patience please... interaction is one of the key aspects of development. Good practice for me and others too.

What about now?

@joshbruce
Copy link
Member

@Martii: I do apologize, I was not becoming impatient, I just honestly didn't want you to feel like you should have to create something, that in all honesty, I (or one of the other core team) probably should have given a template for. :)

LGTM!

@joshbruce joshbruce requested a review from styfle April 20, 2018 02:05
@joshbruce
Copy link
Member

@Martii: One the other settings we have is that it takes two approvers before getting merged. So, someone else should be along. If they approve, they can merge and do all the other things. Thanks for being our first "registered" user! Very exciting stuff.

Also, I'm curious about your (apparent) fascination with legal things.

@Martii
Copy link
Contributor Author

Martii commented Apr 20, 2018

I am quite flexible when it comes to changes but I am also old school and have a telephone to attend too sometimes. ;) I also enjoy open source development... which is why I'm here. None of us gets paid (well usually not) so I prefer to be as polite as I can be.

I'm curious about your (apparent) fascination with legal things

Because I'm a non-attorney (which may change someday) advocate. I prefer to keep things neat and organized as possible... although sometimes that takes longer than usual in my day to day activities.


Btw I am a small contributor directly to node as they are my favourite Company/Corp. :)

@Martii
Copy link
Contributor Author

Martii commented Apr 20, 2018

Not sure how picky column alignment is in the source... Usually I would just not match the headers and separators to the item table columns/rows... so they are currently misaligned compared to rest but seems trivial I think (hope)

@styfle styfle merged commit 1000d92 into markedjs:master Apr 20, 2018
@Martii Martii deleted the UserShip-Addition branch April 20, 2018 12:52
zhenalexfan pushed a commit to zhenalexfan/MarkdownHan that referenced this pull request Nov 8, 2021
Add self from OUJS Organization to AUTHORS.md
zhenalexfan pushed a commit to zhenalexfan/MarkdownHan that referenced this pull request Nov 8, 2021
* Unfortunately @joshbruce lost some credibility by a post edit with an edit misusing the OpenUserJS brand so respectfully removing listing.

NOTE:
* This is probably why no-one else has done this

Ref:
* markedjs#1233

with

> LGTM!

... apparently it wasn't and shouldn't have been merged without all parties consent.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: docs Documentation changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants