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

Selectable timestamp bracket format #132

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

Royaljerry
Copy link

Hi team, just enhanced this extension a bit, in order to have a selectable timestamp bracket format. Since I use Beorg app on my iPhone, and it uses the <YYYY-MM-DD NoD> instead of the avalable [YYYY-MM-DD NoD], I wanted to have this feature. :)

Best,

Adam

@sipi
Copy link
Member

sipi commented Jan 23, 2019

Hi @Royaljerry,
thanks for this pull request, I will review it as soon as possible.
6pi

@Royaljerry
Copy link
Author

Royaljerry commented Jan 23, 2019 via email

@LoveSponge
Copy link

This is a really nice implementation & adds a degree of customisability to the extension. Looking forward to the merging ☺

Copy link
Member

@sipi sipi left a comment

Choose a reason for hiding this comment

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

I just had a quick look, all seems to be ok. But I haven't yet run it.

@@ -14,7 +14,7 @@
"vscode": "^1.23.0"
},
"categories": [
"Languages"
"Programming Languages"
Copy link
Member

Choose a reason for hiding this comment

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

Probably a good change but not in this PR ;)

@@ -8,22 +8,22 @@

// A task runner that calls a custom npm script that compiles the extension.
{
"version": "0.1.0",
"version": "2.0.0",
Copy link
Member

Choose a reason for hiding this comment

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

Probably a good change but not in this PR ;)


// we want to run npm
"command": "npm",

// the command is a shell script
"isShellCommand": true,
"type": "shell",
Copy link
Member

Choose a reason for hiding this comment

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

Probably a good change but not in this PR ;)


// show the output window only if unrecognized errors occur.
"showOutput": "silent",
"reveal": "silent",
Copy link
Member

Choose a reason for hiding this comment

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

Probably a good change but not in this PR ;)


// we run the custom script "compile" as defined in package.json
"args": ["run", "compile", "--loglevel", "silent"],

// The tsc compiler is started in watching mode
"isWatching": true,
"isBackground": true,
Copy link
Member

Choose a reason for hiding this comment

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

Probably a good change but not in this PR ;)

@@ -2,7 +2,7 @@
"name": "org-mode",
"displayName": "Org Mode",
"description": "Emacs Org mode support for VSCode",
"version": "0.1.3",
"version": "0.5.1",
Copy link
Member

Choose a reason for hiding this comment

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

A good change but not in this PR ;)

@@ -1,6 +1,6 @@
// A launch configuration that compiles the extension and then opens it inside a new window
{
"version": "0.1.0",
"version": "2.0.0",
Copy link
Member

Choose a reason for hiding this comment

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

Probably a good change but not in this PR ;)

Copy link
Member

@AdrieanKhisbe AdrieanKhisbe left a comment

Choose a reason for hiding this comment

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

LGTM but I concour with @sipi comments.
Also, we should wait that the change behvaior preseing [] works. :)

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.

None yet

4 participants