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

Updating to master branch #1

Open
wants to merge 6,434 commits into
base: master-v1.4.2
Choose a base branch
from
Open

Conversation

jkim485
Copy link

@jkim485 jkim485 commented Jun 30, 2017

What? Why?

Fix # .

Changes proposed in this pull request:

  • a
  • b

Checks

  • Runs on Heroku.
  • Follows 7 rules of great commit messages. For most PRs a single commit should suffice, in some cases multiple topical commits can be useful. During review it is ok to see tiny commits (e.g. Fix reviewer comments), but right before the code gets merged to master or rc branch, any such commits should be squashed since they are useless to the other developers. Definitely avoid merge commits, use rebase instead.
  • Follows the Google Style Guide.
  • Make sure your commit messages end with a Signed-off-by string (this line
    can be automatically added by git if you run the git-commit command with
    the -s option)
  • If this is a feature, the PR is to rc. If this is a bug fix, the PR is to
    hotfix.

Any screenshots or GIFs?

If this is a new visual feature please add a before/after screenshot or gif
here with e.g. GifGrabber.

Notify reviewers

Read our Pull request merging
policy
. If you are part of the
cBioPortal organization, notify the approprate team (remove inappropriate):

@cBioPortal/frontend
@cBioPortal/backend
@cBioPortal/devops

If you are not part of the cBioPortal organization look at who worked on the
file before you. Please use git blame <filename> to determine that
and notify them here:

inodb and others added 30 commits March 7, 2024 16:32
…n Types (#10665)

* replace .toList with .collect(Collectors.toList())

* merge nested if statements

* refactor codes from community call

---------

Co-authored-by: Qi-Xuan Lu <[email protected]>
Co-authored-by: Jeremy R. Easton-Marks <[email protected]>
…ation

Replace deprecated Spring Security class
🐛 Update core version in dockerFile
➖ Remove mockserver dep from production build
* Update Saml Security Tests

* 🔧 Update configs for integration tests

* 🔧 Fix docker-entrypoint to support new spring db props

* Add docker_image to test integration test

* Add Authenticate=false to application.properties

* Test updating PORTAL_HOME

* Revert authenticate property
Fix: groups and authorities filter issue
…10728)

* 🐛 Fix RedisAutoConfig Exclusion logic to exclude if redis session host is set

* Update property to be spring.session.store-type
inodb and others added 30 commits January 7, 2025 10:10
* consolidate api tests and security tests
* update config version to 2.1
* use local image for security tests
* skip pulling image for docker scout
* remove dependency on frontend code
* Input validation
* squash bins
Improve circleci tests with reusable docker images
Add new properties for server url and key

Co-authored-by: Avery Wang <[email protected]>
Improved logging for security tests on circleci
* upgrade to spring boot 3.4

* fix critical and high vulnerabilities

* fix vulnerability with commons-io

* fix mysql connector vulnerability

* fix security integration tests

* add back test scopes

* remove conflicting dependencies

---------

Co-authored-by: Bryan Lai <[email protected]>
* add migration guide for v6.0.24

---------

Co-authored-by: Ino de Bruijn <[email protected]>
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.