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

Makes minor improvements to PR-294 #344

Conversation

prasadtalasila
Copy link
Contributor

Fixes some markdown linting issues in PR #294

lindot11 and others added 30 commits October 3, 2023 13:36
lindot11 and others added 23 commits November 22, 2023 16:49
  - Adds package publication for runner
  - Updates READMEs of lib and runner microservices
  - Cleans up yarn commands of runner to make them cross-platform
  - Adds documentation for new example
removed typo

added repo to test pre-commit

test pre-commit with typo

undo typo and uncomment mdlint

test pre-commit with typo

remove type

test pre-commit

uncomment format-check

test dummy format

test dummy

commit dummies to git for test git diff

test pre-commit eslint

pre-commit config to replace custom pre-commit

added directory lib to hooks

test pre-commit yarn install

yarn install logic to pre-commit

prettified pre-commit-config

added pre-push hooks

test dummy pre-push

test pre-push yarn jest

test pre-push yarn jest

test again

test pre-push again

test again

TEST CORRECTLY

test manual push

another manual test

test all pre-commit hooks

added default hooks

added pre-push test for lib

test git hooks dir

test git hooks dir

test dir with dummie

undo hook dir back to before

test git hooks

test git hooks

test git hooks

test git hooks

test git hooks

test git hooks

test git hooks

test git hooks

test git hooks

test git hooks

test git hooks

test git hooks

reset after test

test git hooks

test git hooks

test git hooks

reset after test

test git hooks

reset after test

test git hooks

test git hooks

reset after test

test git hooks

reset after test

reset after test

reset after test

reset after test

reset after test

test script for git hooks done

githook test and log for documentation

reset after test

reset after test

reset after test

reset after test

reset after test

reset after test

reset after test

reset after test

test git hooks

reset after test

test git hooks

reset after test

test git hooks

reset after test

improved test

went back to original test

test git hooks

final test script

reset after test

test git hooks

reset after test

reset after test

reset after test

reset after test

reset after test

test and log done

added new githook script and updated guide

comment out lib test hook

stage markdownlint changes

log githook run

accept prettified changes

updated run log

removed logs and test from repo

updated configure-git-hooks script and docs

updated hook to use markdownlint/markdownlint

removed unnecessary readme

added prettier config to client

undo change

non trailling commas for client and lib

undo trailing commas to test code cov

Revert "undo trailing commas to test code cov"

This reverts commit e5bf319.

undo trailling commas to test codecov

undo trailling commas in lib

undo single quote prettier

test undo semiconlons

test remove trailling commas lib

set traillingcomma all

undo prettierrc

accept prettier lib improves codecov

accept prettier client improves codecov

Swap ml-workspace container and improve docs (#300)

  - Replaces ml-workspace container with
    ml-workspace-minimal container. This change
    potentially improves the speed of user
    workspace.
  - Adds guide book to help with non-standard
    installation scenarios.
  - Adds MongoDB installation to services.js

---------
Co-authored-by: nichlaes <[email protected]>

Corrects docker commands and updates docs (#304)

  - Corrects docker run commands to always restart the containers
  - Updates diagrams in docs to include MongoDB
  - Adds yarn clean command to package.json of services.

uncomment prettier

replace deprecated code_blocks

uncomment markdownlint

fixed table mdl error

ignore url line length mdl

redo  mdl hook

ignore table line length and use .markdownlint.yaml instead

replaced by pre-commit framework

adhere to code climate linelength

Adds scope to library tabs (#235 and #321)

  - Shows the right directory for each tab in Library page
  - Adds second level tabs to Library page. These tabs separate
     private and common assets

---------
Co-authored-by: caesarv16 <[email protected]>
Co-authored-by: s88004 <[email protected]>

markdownlint disable codeblock test

test deprecated code_blocks in codeclimate

shortened comment to adhere to line_length
author lindot11 <[email protected]> 1701026896 +0100
committer Linda Nguyen <[email protected]> 1701283986 +0100

parent 7709e00
author lindot11 <[email protected]> 1701026896 +0100
committer Linda Nguyen <[email protected]> 1701283967 +0100

parent 7709e00
author lindot11 <[email protected]> 1701026896 +0100
committer Linda Nguyen <[email protected]> 1701283901 +0100

parent 7709e00
author lindot11 <[email protected]> 1701026896 +0100
committer Linda Nguyen <[email protected]> 1701283878 +0100

parent 7709e00
author lindot11 <[email protected]> 1701026896 +0100
committer Linda Nguyen <[email protected]> 1701283866 +0100

parent 7709e00
author lindot11 <[email protected]> 1701026896 +0100
committer Linda Nguyen <[email protected]> 1701283821 +0100

parent 7709e00
author lindot11 <[email protected]> 1701026896 +0100
committer Linda Nguyen <[email protected]> 1701283802 +0100

parent 7709e00
author lindot11 <[email protected]> 1701026896 +0100
committer Linda Nguyen <[email protected]> 1701283787 +0100

parent 7709e00
author lindot11 <[email protected]> 1701026896 +0100
committer Linda Nguyen <[email protected]> 1701283759 +0100

skip package upgrade

added new githook script and updated guide

comment out lib test hook

log githook run

updated run log

removed logs and test from repo

updated configure-git-hooks script and docs

removed unnecessary readme

added prettier config to client

undo change

non trailling commas for client and lib

undo trailing commas to test code cov

Revert "undo trailing commas to test code cov"

This reverts commit e5bf319.

test undo semiconlons

set traillingcomma all

undo prettierrc

accept prettier lib improves codecov

Corrects docker commands and updates docs (#304)

  - Corrects docker run commands to always restart the containers
  - Updates diagrams in docs to include MongoDB
  - Adds yarn clean command to package.json of services.

uncomment prettier

replaced by pre-commit framework

adhere to code climate linelength

markdownlint disable codeblock test

test deprecated code_blocks in codeclimate

shortened comment to adhere to line_length

skip package upgrade

Update docs and adds publication for runner (#275)

  - Adds package publication for runner
  - Updates READMEs of lib and runner microservices
  - Cleans up yarn commands of runner to make them cross-platform
  - Adds documentation for new example

  Fixes mistakes in the docs

test git hooks

test git hooks

test git hooks

test git hooks

test git hooks

test git hooks

test git hooks

test git hooks

test git hooks

test git hooks

test git hooks

test git hooks

reset after test

test git hooks

test git hooks

test git hooks

reset after test

test git hooks

reset after test

test git hooks

test git hooks

reset after test

test git hooks

reset after test

reset after test

reset after test

reset after test

reset after test

test script for git hooks done

githook test and log for documentation

reset after test

reset after test

reset after test

reset after test

reset after test

reset after test

reset after test

reset after test

test git hooks

reset after test

test git hooks

reset after test

test git hooks

reset after test

improved test

went back to original test

test git hooks

final test script

reset after test

test git hooks

reset after test

reset after test

reset after test

reset after test

reset after test

test and log done

added new githook script and updated guide

comment out lib test hook

log githook run

updated run log

removed logs and test from repo

updated configure-git-hooks script and docs

updated hook to use markdownlint/markdownlint

undo change

non trailling commas for client and lib

undo trailing commas to test code cov

Revert "undo trailing commas to test code cov"

This reverts commit e5bf319.

undo trailling commas to test codecov

undo trailling commas in lib

undo single quote prettier

test undo semiconlons

test remove trailling commas lib

set traillingcomma all

undo prettierrc

accept prettier lib improves codecov

accept prettier client improves codecov

Swap ml-workspace container and improve docs (#300)

  - Replaces ml-workspace container with
    ml-workspace-minimal container. This change
    potentially improves the speed of user
    workspace.
  - Adds guide book to help with non-standard
    installation scenarios.
  - Adds MongoDB installation to services.js

---------
Co-authored-by: nichlaes <[email protected]>

Corrects docker commands and updates docs (#304)

  - Corrects docker run commands to always restart the containers
  - Updates diagrams in docs to include MongoDB
  - Adds yarn clean command to package.json of services.

uncomment prettier

replace deprecated code_blocks

uncomment markdownlint

fixed table mdl error

ignore url line length mdl

redo  mdl hook

ignore table line length and use .markdownlint.yaml instead

markdownlint disable codeblock test

fix wrong reset action

Revert "accept prettier client improves codecov"

This reverts commit 924a5cd.

Revert "skip package upgrade"

This reverts commit ae8bea4.

```bibtex
@misc{talasila2023digital,
title={Digital Twin as a Service (DTaaS): A Platform for Digital Twin Developers and Users},
Copy link

Choose a reason for hiding this comment

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

Trailing spaces

Copy link

codeclimate bot commented Dec 3, 2023

Code Climate has analyzed commit 8a3f5ec and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Style 1

View more on Code Climate.

@codecov-commenter
Copy link

codecov-commenter commented Dec 3, 2023

Codecov Report

Merging #344 (8a3f5ec) into feature/distributed-demo (0631410) will increase coverage by 1.15%.
Report is 2 commits behind head on feature/distributed-demo.
The diff coverage is 50.00%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@                     Coverage Diff                      @@
##           feature/distributed-demo     #344      +/-   ##
============================================================
+ Coverage                     63.47%   64.63%   +1.15%     
============================================================
  Files                            31       43      +12     
  Lines                           397      574     +177     
  Branches                         26       35       +9     
============================================================
+ Hits                            252      371     +119     
- Misses                          130      180      +50     
- Partials                         15       23       +8     
Files Coverage Δ
client/src/components/tab/TabComponent.tsx 88.88% <ø> (ø)
client/src/route/library/Library.tsx 100.00% <ø> (ø)
servers/lib/src/files/resolvers/files.resolver.ts 81.81% <100.00%> (ø)
...ers/lib/src/files/services/gitlab-files.service.ts 95.23% <100.00%> (ø)
servers/lib/src/types.ts 84.21% <100.00%> (ø)
...rs/lib/src/files/services/files-service.factory.ts 71.42% <66.66%> (ø)
servers/lib/src/files/files.module.ts 0.00% <0.00%> (ø)
servers/lib/src/bootstrap.ts 0.00% <0.00%> (ø)
...vers/lib/src/files/services/local-files.service.ts 22.22% <30.76%> (ø)

... and 5 files with indirect coverage changes

Components Coverage Δ
Website 63.47% <ø> (ø)
Lib Microservice 58.95% <50.00%> (∅)

@prasadtalasila prasadtalasila merged commit 3080647 into INTO-CPS-Association:feature/distributed-demo Dec 3, 2023
6 checks passed
This was referenced Dec 3, 2023
@prasadtalasila prasadtalasila deleted the pr-294 branch December 4, 2023 12:03
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.

5 participants