Skip to content

config(EJ2-5374): Modified sample browser layout #1

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

Merged
merged 4 commits into from
Nov 30, 2017
Merged

config(EJ2-5374): Modified sample browser layout #1

merged 4 commits into from
Nov 30, 2017

Conversation

vairamuthuR
Copy link
Contributor

publish java script sample browser

debug.log Outdated
@@ -0,0 +1,2 @@
[1128/175427:ERROR:tcp_listen_socket.cc(76)] Could not bind socket to 127.0.0.1:6004
[1128/175427:ERROR:node_debugger.cc(86)] Cannot start debugger server
Copy link
Contributor

Choose a reason for hiding this comment

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

remove this file

config.json Outdated
@@ -0,0 +1,9 @@
{
"htmllint": ["./src/**/*.html"],
Copy link
Contributor

Choose a reason for hiding this comment

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

remove this file


```
npm run build
```
Copy link
Contributor

Choose a reason for hiding this comment

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

Add running comment, refer the below ReadMe for further details.

https://github.com/syncfusion/ej2-samples/#running

gulpfile.js Outdated
var config =require(fs.realpathSync('./config.json'));

var curDirectory = '';
//var build = require("@syncfusion/ej2-build");
Copy link
Contributor

Choose a reason for hiding this comment

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

remove this line

package.json Outdated
"jshint": "^2.9.2"
},
"scripts": {
"test": "gulp js-hint && gulp build",
Copy link
Contributor

Choose a reason for hiding this comment

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

build script is missing, but added in the ReadMe file

package.json Outdated
"serve": "gulp serve"
},

"config": {
Copy link
Contributor

Choose a reason for hiding this comment

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

remove this config

@ajithr ajithr merged commit e861fd7 into syncfusion:master Nov 30, 2017
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