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

Review #3

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Review #3

wants to merge 8 commits into from

Conversation

earlAchromatic
Copy link
Owner

@earlAchromatic earlAchromatic commented Apr 22, 2022

Dummy commit for reviewable app


This change is Reviewable

@@ -1,4 +1,4 @@
# Pragmatic Vue Components 🔌
# Pragmatic Vue Components

This comment was marked as off-topic.

Copy link
Owner Author

Choose a reason for hiding this comment

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

Nice title

This comment was marked as off-topic.

@@ -13,19 +13,16 @@ const componentRegistry = {

module.exports = function (eleventyConfig) {
eleventyConfig.addPassthroughCopy({ public: '/' });
eleventyConfig.addPassthroughCopy('components');
// eleventyConfig.addPassthroughCopy('components');

eleventyConfig.addPassthroughCopy('node_modules');

eleventyConfig.on(
'eleventy.after',
async ({ dir, results, runMode, outputMode }) => {
Copy link
Owner Author

Choose a reason for hiding this comment

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

Is this a good change?

This comment was marked as abuse.

Copy link
Owner Author

Choose a reason for hiding this comment

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

Not a great change...

@earlAchromatic
Copy link
Owner Author

top level comment

@earlAchromatic

This comment was marked as off-topic.

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

How about this?

Reviewable status: 0 of 6 files reviewed, 2 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

another comment.

Reviewable status: 0 of 6 files reviewed, 2 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

one more try here...

Reviewable status: 0 of 6 files reviewed, 2 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

Keep making comms

Reviewable status: 0 of 6 files reviewed, 2 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

how

Reviewable status: 0 of 6 files reviewed, 2 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

asgga

Reviewable status: 0 of 6 files reviewed, 2 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

sgdsag

Reviewable status: 0 of 6 files reviewed, 2 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

saljhklsahg

Reviewable status: 0 of 6 files reviewed, 2 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

Here is a comment for you

Reviewable status: 0 of 6 files reviewed, 2 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

Another nice comment.

Reviewable status: 0 of 6 files reviewed, 2 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

Feedback is great.

Reviewable status: 0 of 6 files reviewed, 2 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

Another Comment Here

Reviewable status: 0 of 6 files reviewed, 2 unresolved discussions

@earlAchromatic
Copy link
Owner Author

@prcl8tr wgat yo

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

Reviewable status: 0 of 6 files reviewed, all discussions resolved


.eleventy.js line 22 at r2 (raw file):

Previously, earlAchromatic (Jacob Milhorn) wrote…

Not a great change...

???

ultimately

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

Reviewed 6 of 6 files at r2, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @earlAchromatic)

@earlAchromatic
Copy link
Owner Author

+@oscl8tr

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

Successfully merging this pull request may close these issues.

2 participants