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

logging to console (behind flag) #8

Merged
merged 4 commits into from
Jan 9, 2019
Merged

logging to console (behind flag) #8

merged 4 commits into from
Jan 9, 2019

Conversation

tpluscode
Copy link
Contributor

@tpluscode tpluscode commented Jan 9, 2019

Fixes #7

@bergos, I added a simple stream which pushes log strings for events with a name. That should be enough for us for now.

Do you have anything else in mind?

@tpluscode tpluscode requested a review from bergos January 9, 2019 11:02
Copy link
Contributor

@bergos bergos left a comment

Choose a reason for hiding this comment

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

Can you move bin/TextLogStream.js to lib/TextLogStream.js? There are no clear rules for the folder structure, but I think usually only the code which is used in the bin property of package.json is in the bin folder.

bin/barnard59.js Outdated Show resolved Hide resolved
@tpluscode tpluscode merged commit c638d2c into master Jan 9, 2019
@tpluscode tpluscode deleted the logging branch January 9, 2019 12:59
tpluscode pushed a commit that referenced this pull request Jun 15, 2023
feat: added buildCubeShape operation
tpluscode pushed a commit that referenced this pull request Jun 20, 2023
tpluscode pushed a commit that referenced this pull request Jun 22, 2023
added concat operation
tpluscode pushed a commit that referenced this pull request Jul 24, 2023
use sparql-http-client + added get operation
tpluscode pushed a commit that referenced this pull request Aug 1, 2023
docs: document package operations in README
tpluscode pushed a commit that referenced this pull request Aug 24, 2023
fix: csvw parser BOM patch
tpluscode pushed a commit that referenced this pull request Aug 31, 2023
fix: only step doesn't need to be readable or writable
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