GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — refactor code to match new maven-based project structure by mainrs · Pull Request #815 · TheAlgorithms/Java

Conversation

@mainrs

@mainrs mainrs commented Aug 24, 2019 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

This commit adds all necessary files as well as correctly specifies the package names for each file. The project now fully mimics maven's project structure (as indicated as a goal earlier in issue #474).

The goal is to make working on the project easier for others. They can now clone the repo and use the Open as maven project that is included in every mainstream IDE/editor.

**Edit: ** All tests pass. Might be a good idea to setup a Travis CI config that will check for tests.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Sven Lechner added 2 commits August 24, 2019 22:47
refactor code to match new maven-based project structure
ad32773 
This commit adds all necessary files as well as correctly specifies the package names for each file. The project now fully mimics maven's project structure (as indicated as a goal earlier in issue #474)
reformat all code
98533d5
@mainrs mainrs mentioned this pull request Aug 25, 2019
Convert the repository into a Java project #474
Closed
yanglbme added a commit that referenced this pull request Aug 27, 2019
@yanglbme
Merge pull request #795 from laingke/Development
3750999 
* Add Maven frameworks support
* close #529 
* close #815
@yanglbme

yanglbme commented Aug 27, 2019

Copy link
Copy Markdown
Member

#795
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@yanglbme yanglbme closed this Aug 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@mainrs @yanglbme
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.