Skip to content

Conversation

@sunil-lakshman
Copy link
Contributor

No description provided.

Copy link
Contributor

@naman-contentstack naman-contentstack left a comment

Choose a reason for hiding this comment

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

Please use the new logger rather than cliux.print

@sunil-lakshman
Copy link
Contributor Author

Please use the new logger rather than cliux.print

@naman-contentstack Used cliux.print in contentstack-config plugin

"dependencies": {
"@contentstack/cli-command": "~1.7.1",
"@contentstack/cli-utilities": "~1.16.0",
"@contentstack/cli-utilities": "~1.17.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

@sunil-lakshman Since we have up updated the minor version of utilities we need to update the patch version of this package

"dependencies": {
"@contentstack/cli-command": "~1.7.1",
"@contentstack/cli-utilities": "~1.16.0",
"@contentstack/cli-utilities": "~1.17.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

@sunil-lakshman Since we have up updated the minor version of utilities we need to update the patch version of this package

"@contentstack/cli-cm-seed": "~1.13.2",
"@contentstack/cli-command": "~1.7.1",
"@contentstack/cli-utilities": "~1.16.0",
"@contentstack/cli-utilities": "~1.17.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

@sunil-lakshman Since we have up updated the minor version of utilities we need to update the patch version of this package

"@oclif/core": "^4.3.0",
"@oclif/plugin-help": "^6.2.28",
"@contentstack/cli-utilities": "~1.16.0",
"@contentstack/cli-utilities": "~1.17.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

@sunil-lakshman Since we have up updated the minor version of utilities we need to update the patch version of this package

"@contentstack/cli-config": "~1.16.2",
"@contentstack/cli-utilities": "~1.16.0",
"@contentstack/cli-config": "~1.17.0",
"@contentstack/cli-utilities": "~1.17.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

@sunil-lakshman Since we have up updated the minor version of utilities and config we need to update the patch version of this package

"dependencies": {
"@contentstack/cli-command": "~1.7.1",
"@contentstack/cli-utilities": "~1.16.0",
"@contentstack/cli-utilities": "~1.17.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

@sunil-lakshman please update the export-to-csv to patch version

@@ -0,0 +1,94 @@
import { Command } from '@contentstack/cli-command';
Copy link
Contributor

Choose a reason for hiding this comment

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

@sunil-lakshman can we please add the debug logs and use the logger that we are using in other config plugins here
cc: @aman19K

@@ -0,0 +1,27 @@
import { Command } from '@contentstack/cli-command';
Copy link
Contributor

Choose a reason for hiding this comment

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

@sunil-lakshman can we please add the debug logs and use the logger that we are using in other config plugins here
cc: @aman19K

@@ -0,0 +1,77 @@
import { Command } from '@contentstack/cli-command';
Copy link
Contributor

Choose a reason for hiding this comment

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

@sunil-lakshman can we please add the debug logs and use the logger that we are using in other config plugins here
cc: @aman19K

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.

4 participants