From fe53dd47476f5952a0a0cf736997b02e68cd920b Mon Sep 17 00:00:00 2001 From: Chris Houseknecht Date: Fri, 10 Aug 2018 03:43:09 -0400 Subject: [PATCH] Revert "Make documentation link more visible. (#1030)" This reverts commit 690d7bca7f0309c14c270537400862f9f08383b6. --- .travis.yml | 5 - AUTHORS | 2 - CONTRIBUTING.rst | 20 -- Makefile | 18 -- galaxy/importer/loaders/apb.py | 34 +- galaxy/importer/tests/test_apb_loader.py | 10 - galaxyui/.prettierrc.yaml | 3 - galaxyui/package.json | 3 +- galaxyui/src/app/app-routing.module.ts | 79 +---- galaxyui/src/app/app.component.html | 25 +- galaxyui/src/app/app.component.spec.ts | 46 +-- galaxyui/src/app/app.component.ts | 143 +++++---- galaxyui/src/app/app.module.ts | 102 +++--- galaxyui/src/app/auth/auth.service.ts | 78 +++-- .../src/app/authors/authors.component.spec.ts | 31 +- galaxyui/src/app/authors/authors.component.ts | 116 ++++--- galaxyui/src/app/authors/authors.module.ts | 41 +-- .../app/authors/authors.resolver.service.ts | 45 ++- .../src/app/authors/authors.routing.module.ts | 54 +++- .../detail/author-detail.component.spec.ts | 31 +- .../authors/detail/author-detail.component.ts | 131 ++++---- .../detail-actions.component.ts | 65 ++-- .../cloud-platforms.component.ts | 19 +- .../dependencies/dependencies.component.ts | 18 +- .../cards/info/card-info.component.ts | 30 +- .../cards/platforms/platforms.component.ts | 20 +- .../cards/versions/versions.component.ts | 21 +- .../content-detail.component.spec.ts | 31 +- .../content-detail.component.ts | 118 +++---- .../content-detail/content-detail.module.ts | 53 ++-- .../content-detail.resolver.service.ts | 51 ++- .../content-detail.routing.module.ts | 48 ++- .../module-utils/module-utils.component.ts | 113 ++++--- .../content/modules/modules.component.ts | 114 ++++--- .../content/plugins/plugins.component.ts | 131 ++++---- .../content/roles/roles.component.ts | 114 ++++--- .../repository/repository.component.ts | 42 ++- galaxyui/src/app/enums/content-types.enum.ts | 11 +- .../src/app/enums/contributor-types.enum.ts | 4 +- galaxyui/src/app/enums/import-state.enum.ts | 2 +- galaxyui/src/app/enums/repo-types.enum.ts | 4 +- galaxyui/src/app/enums/view-types.enum.ts | 8 +- .../access-denied/access-denied.component.ts | 51 +-- .../exception-pages/exception-pages.module.ts | 23 +- .../exception-pages.routing.module.ts | 21 +- .../not-found/not-found.component.ts | 51 +-- .../app/home/carousel/carousel.component.ts | 24 +- galaxyui/src/app/home/home.component.spec.ts | 31 +- galaxyui/src/app/home/home.component.ts | 38 ++- galaxyui/src/app/home/home.module.ts | 42 ++- .../src/app/home/home.resolver.service.ts | 35 +- galaxyui/src/app/home/home.routing.module.ts | 39 ++- .../src/app/home/popular/popular.component.ts | 50 +-- .../src/app/login/login.component.spec.ts | 31 +- galaxyui/src/app/login/login.component.ts | 46 ++- galaxyui/src/app/login/login.module.ts | 29 +- .../src/app/login/login.routing.module.ts | 31 +- .../add-repository-modal.component.spec.ts | 31 +- .../add-repository-modal.component.ts | 94 +++--- .../src/app/my-content/my-content.module.ts | 65 ++-- .../my-content/my-content.routing.module.ts | 59 ++-- .../namespace-detail.component.spec.ts | 31 +- .../namespace-detail.component.ts | 163 +++++----- .../namespace-detail.resolver.service.spec.ts | 16 +- .../namespace-detail.resolver.service.ts | 52 +-- .../namespace-list/action/action.component.ts | 108 ++++--- .../owners-content.component.spec.ts | 31 +- .../owners-content.component.ts | 16 +- ...vider-namespaces-content.component.spec.ts | 31 +- .../provider-namespaces-content.component.ts | 16 +- .../action/action.component.ts | 71 +++-- .../alternate-name-modal.component.spec.ts | 33 +- .../alternate-name-modal.component.ts | 57 ++-- .../repositories-content.component.spec.ts | 31 +- .../repositories-content.component.ts | 127 ++++---- .../namespace-list-resolver.service.spec.ts | 16 +- .../namespace-list-resolver.service.ts | 26 +- .../namespace-list.component.spec.ts | 31 +- .../namespace-list.component.ts | 119 +++---- .../import-detail.component.spec.ts | 31 +- .../import-detail/import-detail.component.ts | 85 +++-- .../import-list/import-list.component.spec.ts | 31 +- .../import-list/import-list.component.ts | 251 ++++++++------- .../import-list.resolver.service.ts | 31 +- .../src/app/my-imports/my-imports.module.ts | 39 ++- .../my-imports/my-imports.routing.module.ts | 33 +- .../page-header/page-header.component.spec.ts | 31 +- .../app/page-header/page-header.component.ts | 15 +- .../src/app/page-header/page-header.module.ts | 19 +- .../page-loading.component.spec.ts | 31 +- .../page-loading/page-loading.component.ts | 13 +- .../app/page-loading/page-loading.module.ts | 16 +- .../api-root/api-root.service.spec.ts | 16 +- .../resources/api-root/api-root.service.ts | 31 +- .../src/app/resources/api-root/api-root.ts | 2 +- .../cloud-platform.service.spec.ts | 16 +- .../cloud-platforms/cloud-platform.service.ts | 53 ++-- .../content-blocks.service.spec.ts | 16 +- .../content-blocks/content-blocks.service.ts | 42 +-- .../content-search.service.spec.ts | 16 +- .../content-search/content-search.service.ts | 51 ++- .../content-type.service.spec.ts | 16 +- .../content-types/content-type.service.ts | 42 ++- .../resources/content/content.service.spec.ts | 16 +- .../app/resources/content/content.service.ts | 72 +++-- .../resources/imports/imports.service.spec.ts | 16 +- .../app/resources/imports/imports.service.ts | 61 ++-- .../namespaces/namespace.service.spec.ts | 16 +- .../resources/namespaces/namespace.service.ts | 57 ++-- .../src/app/resources/namespaces/namespace.ts | 4 +- .../app/resources/pf-body/pf-body.service.ts | 36 ++- .../platforms/platform.service.spec.ts | 16 +- .../resources/platforms/platform.service.ts | 52 +-- .../provider-source.service.spec.ts | 16 +- .../provider-source.service.ts | 41 ++- .../repositories/repository.service.spec.ts | 16 +- .../repositories/repository.service.ts | 66 ++-- .../repository-import.service.spec.ts | 16 +- .../repository-import.service.ts | 46 +-- .../app/resources/tags/tags.service.spec.ts | 16 +- .../src/app/resources/tags/tags.service.ts | 52 +-- .../app/resources/users/user.service.spec.ts | 16 +- .../src/app/resources/users/user.service.ts | 33 +- .../search/popular/popular.component.spec.ts | 31 +- .../app/search/popular/popular.component.ts | 44 ++- .../src/app/search/search-routing.module.ts | 37 ++- .../src/app/search/search.component.spec.ts | 31 +- galaxyui/src/app/search/search.component.ts | 299 ++++++++++-------- galaxyui/src/app/search/search.module.ts | 39 +-- .../src/app/search/search.resolver.service.ts | 89 ++++-- .../user-notifications.component.spec.ts | 31 +- .../user-notifications.component.ts | 18 +- .../clipboard/clipboard.component.ts | 16 +- .../src/app/utilities/utilities.module.ts | 24 +- .../app/vendors/card/vendor-card.component.ts | 26 +- galaxyui/src/app/vendors/vendors.component.ts | 110 ++++--- galaxyui/src/app/vendors/vendors.module.ts | 38 ++- .../app/vendors/vendors.resolver.service.ts | 25 +- .../src/app/vendors/vendors.routing.module.ts | 47 ++- galaxyui/src/environments/environment.prod.ts | 2 +- galaxyui/src/environments/environment.ts | 2 +- galaxyui/src/main.ts | 7 +- galaxyui/src/polyfills.ts | 8 +- galaxyui/src/test.ts | 12 +- galaxyui/src/typings.d.ts | 2 +- galaxyui/tslint.json | 37 ++- galaxyui/yarn.lock | 4 - scripts/docker/dev/start-develop.sh | 1 + 148 files changed, 3661 insertions(+), 2710 deletions(-) delete mode 100644 galaxyui/.prettierrc.yaml diff --git a/.travis.yml b/.travis.yml index 77abe8970..af8238c8e 100644 --- a/.travis.yml +++ b/.travis.yml @@ -28,7 +28,6 @@ script: - make dev/waitenv - make dev/test - # check images running - docker ps -a - docker images @@ -39,10 +38,6 @@ script: - ansible-playbook -i inventory test.yml - popd -# Run JavaScript test -- make dev/jslint -- make test/prettier - # Check the logs - docker ps -a - make dev/log diff --git a/AUTHORS b/AUTHORS index 164621e86..b1ad4ba9c 100644 --- a/AUTHORS +++ b/AUTHORS @@ -8,8 +8,6 @@ Chris Church Chris Houseknecht Christopher Chase David Newswanger -David Zager -Ivan Remizov James Cammarata Jiri Tyr Joe Fiorini diff --git a/CONTRIBUTING.rst b/CONTRIBUTING.rst index f01a54973..7034b8734 100644 --- a/CONTRIBUTING.rst +++ b/CONTRIBUTING.rst @@ -430,26 +430,6 @@ To run lint checks against JavaScript/TypeScript sources, execute: $ make dev/jslint -Formatting your code -^^^^^^^^^^^^^^^^^^^^ - -We use prettier to enforce code formatting for all of our TypeScript and less files. -To automatically format your Angular code run: - -.. code-block:: console - - $ make dev/prettier - -Unformatted code will cause the travis build to fail when you push your changes to -GitHub. - -It's recommended that you set up prettier on your editor if you're making lots of -changes to anything in ``galaxyui/``. Prettier is supported by most major editors -and you can find more `information about that here `_. - -Our prettier configuration can be found at ``galaxyui/.prettierrc.yaml``. Please -use it when setting up your editor. - Testing your code ^^^^^^^^^^^^^^^^^ diff --git a/Makefile b/Makefile index 9520134e0..f6f21c6f5 100644 --- a/Makefile +++ b/Makefile @@ -98,13 +98,6 @@ test/flake8: test/jslint: cd galaxyui; ng lint -# for some reason yarn won't add prettier to /usr/local/bin -.PHONY: test/prettier -test/prettier: - @echo "Checking formatting..." - @echo "If this fails, run make dev/prettier locally to format your code" - cd galaxyui; ./node_modules/.bin/prettier -l "{src/**/*.ts, src/**/*.less}" - # --------------------------------------------------------- # Docker targets # --------------------------------------------------------- @@ -151,17 +144,6 @@ dev/jslint: @echo "Linting Javascript..." @$(DOCKER_COMPOSE) exec galaxy bash -c 'cd galaxyui; ng lint' -# for some reason yarn won't add prettier to /usr/local/bin -.PHONY: dev/prettier -dev/prettier: - @echo "Running prettier..." - @$(DOCKER_COMPOSE) exec galaxy bash -c 'cd galaxyui; ./node_modules/.bin/prettier --write "{src/**/*.ts, src/**/*.less}"' - -.PHONY: dev/shellcheck -dev/shellcheck: - @$(DOCKER_COMPOSE) exec galaxy bash -c '\ - find ./scripts -name *.sh | xargs shellcheck' - .PHONY: dev/test dev/test: @echo "Running tests" diff --git a/galaxy/importer/loaders/apb.py b/galaxy/importer/loaders/apb.py index d7cf8246b..fd1f86aef 100644 --- a/galaxy/importer/loaders/apb.py +++ b/galaxy/importer/loaders/apb.py @@ -33,18 +33,6 @@ class APBMetaParser(object): # Tags should contain lowercase letters and digits only TAG_REGEXP = re.compile('^[a-z0-9]+$') - # APB parameters should be in json-schema form - PARAM_KEY_MAP = { - 'maxlength': 'maxLength', - 'max_length': 'maxLength', - 'min_length': 'minLength', - 'multiple_of': 'multipleOf', - 'exclusive_maximum': 'exclusiveMaximum', - 'exclusive_minimum': 'exclusiveMinimum', - 'display_type': 'displayType', - 'display_group': 'displayGroup' - } - def __init__(self, metadata, logger=None): self.metadata = metadata self.log = logger or base.default_logger @@ -112,13 +100,9 @@ def _check_plans(self): raise exc.APBContentLoadError( 'Expecting "plans" in metadata to be a list') - expected_plan_keys = ('description', - 'free', - 'metadata', - 'bindable', - 'parameters') + expected_plan_keys = ('description', 'free', 'metadata', 'parameters') expected_plan_meta_keys = ('displayName', 'longDescription', 'cost') - expected_parameter_keys = ('name', 'title', 'type', 'required') + expected_parameter_keys = ('name', 'title', 'type') idx = 0 for plan in plans: if not isinstance(plan, dict): @@ -153,13 +137,6 @@ def _check_plans(self): expected_parameter_keys, 'plans[{0}].parameters[{1}]'.format(idx, pidx), params) - for param_key in params.keys(): - if param_key in self.PARAM_KEY_MAP: - new_key = self.PARAM_KEY_MAP[param_key] - self.metadata[fieldname][idx]['parameters'][pidx][ - new_key] = self.metadata[fieldname][idx][ - 'parameters'][pidx].pop(param_key) - pidx += 1 idx += 1 @@ -190,9 +167,6 @@ def _validate_tag(self, tag): return False return True - def parse_metadata(self): - return self.metadata - def parse_tags(self): tags = [] apb_tags = self.metadata.get('tags', []) @@ -214,11 +188,11 @@ def __init__(self, content_type, path, root, metadata_path, logger=None): def load(self): self.log.info('Loading metadata file: {0}'.format(self.metadata_file)) - meta_parser = APBMetaParser(self._load_metadata(), logger=self.log) + metadata = self._load_metadata() + meta_parser = APBMetaParser(metadata, logger=self.log) name = meta_parser.parse_name() description = meta_parser.parse_description() meta_parser.check_data() - metadata = meta_parser.parse_metadata() data = {'tags': meta_parser.parse_tags()} readme = self._get_readme() diff --git a/galaxy/importer/tests/test_apb_loader.py b/galaxy/importer/tests/test_apb_loader.py index 40b4f3731..5ac4df832 100644 --- a/galaxy/importer/tests/test_apb_loader.py +++ b/galaxy/importer/tests/test_apb_loader.py @@ -327,16 +327,6 @@ def test_missing_version(self): msg = 'Missing "version" field in metadata.' assert msg in excinfo.value.message - def test_param_keys(self): - parser = apb_loader.APBMetaParser(self.data, self.log) - parser.check_data() - metadata = parser.parse_metadata() - assert 'displayGroup' in metadata['plans'][0]['parameters'][0] - assert 'displayType' in metadata['plans'][0]['parameters'][1] - assert 'displayGroup' in metadata['plans'][0]['parameters'][1] - assert 'minLength' in metadata['plans'][0]['parameters'][1] - assert 'maxLength' in metadata['plans'][0]['parameters'][1] - def test_version_format(self): self.data['version'] = 'foo' parser = apb_loader.APBMetaParser(self.data, self.log) diff --git a/galaxyui/.prettierrc.yaml b/galaxyui/.prettierrc.yaml deleted file mode 100644 index 582a04ef4..000000000 --- a/galaxyui/.prettierrc.yaml +++ /dev/null @@ -1,3 +0,0 @@ -printWidth: 140 -singleQuote: true -trailingComma: "all" diff --git a/galaxyui/package.json b/galaxyui/package.json index bd9d5dd3f..6da606699 100644 --- a/galaxyui/package.json +++ b/galaxyui/package.json @@ -1,7 +1,7 @@ { "name": "galaxyui", "version": "0.0.0", - "license": "Apache-2.0", + "license": "MIT", "scripts": { "ng": "ng", "start": "ng serve", @@ -43,7 +43,6 @@ "karma-coverage-istanbul-reporter": "^1.2.1", "karma-jasmine": "~1.1.0", "karma-jasmine-html-reporter": "^0.2.2", - "prettier": "1.14.0", "protractor": "~5.1.2", "ts-node": "~4.1.0", "tslint": "~5.9.1", diff --git a/galaxyui/src/app/app-routing.module.ts b/galaxyui/src/app/app-routing.module.ts index 36e9679cc..171c255bb 100644 --- a/galaxyui/src/app/app-routing.module.ts +++ b/galaxyui/src/app/app-routing.module.ts @@ -1,82 +1,25 @@ -import { NgModule } from '@angular/core'; -import { AuthorDetailComponent } from './authors/detail/author-detail.component'; -import { ContentDetailComponent } from './content-detail/content-detail.component'; -import { NotFoundComponent } from './exception-pages/not-found/not-found.component'; - -import { - ContentResolver, - NamespaceResolver, - RepositoryResolver as ContentRepositoryResolver, -} from './content-detail/content-detail.resolver.service'; - -import { NamespaceDetailResolver, RepositoryResolver as AuthorRepositoryResolver } from './authors/authors.resolver.service'; - -import { PreloadAllModules, RouterModule, Routes } from '@angular/router'; +import { NgModule } from '@angular/core'; +import { RouterModule, Routes } from '@angular/router'; +import { NotFoundComponent } from './exception-pages/not-found/not-found.component'; const appRoutes: Routes = [ { path: '', redirectTo: '/home', - pathMatch: 'full', - }, - - // Lazily loaded modules - { - path: 'search', - loadChildren: './search/search.module#SearchModule', - }, - { - path: 'my-content', - loadChildren: './my-content/my-content.module#MyContentModule', - }, - { - path: 'my-imports', - loadChildren: './my-imports/my-imports.module#MyImportsModule', - }, - - // Routes that resolve variables have to go in app-routing.module to ensure - // that they are resolved between the static routes ('/search', '/my-content' etc) - // and the wildcard ('**') - { - path: ':namespace/:repository/:content_name', - component: ContentDetailComponent, - resolve: { - content: ContentResolver, - repository: ContentRepositoryResolver, - namespace: NamespaceResolver, - }, - }, - { - path: ':namespace/:repository', - component: ContentDetailComponent, - resolve: { - content: ContentResolver, - repository: ContentRepositoryResolver, - namespace: NamespaceResolver, - }, - }, - { - path: ':namespace', - component: AuthorDetailComponent, - resolve: { - namespace: NamespaceDetailResolver, - repositories: AuthorRepositoryResolver, - }, - }, - { + pathMatch: 'full' + }, { path: '**', - component: NotFoundComponent, - }, + component: NotFoundComponent + } ]; @NgModule({ imports: [ RouterModule.forRoot(appRoutes, { enableTracing: false, - onSameUrlNavigation: 'reload', - preloadingStrategy: PreloadAllModules, - }), + onSameUrlNavigation: 'reload' + }) ], - exports: [RouterModule], + exports: [ RouterModule ] }) -export class AppRoutingModule {} +export class AppRoutingModule { } diff --git a/galaxyui/src/app/app.component.html b/galaxyui/src/app/app.component.html index 1eb04eedc..2355877cb 100644 --- a/galaxyui/src/app/app.component.html +++ b/galaxyui/src/app/app.component.html @@ -11,27 +11,32 @@ [updateActiveItemsOnClick]="false">