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.

Sign up for GitHub

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

Jump to bottom

NEWS doesn't reflect actual changes to data files. #67

Closed
gfinak opened this issue Mar 14, 2019 · 0 comments
Closed

NEWS doesn't reflect actual changes to data files. #67

gfinak opened this issue Mar 14, 2019 · 0 comments
Assignees
@gfinak
Labels
0.16 bug

Comments

@gfinak
Copy link
Member

gfinak commented Mar 14, 2019

Expected behavior: When rebuilding a package and a data set changes, the NEWS file is updated to reflect only the changing data set.

Observed behavior: Sometimes the news file indicates that multiple data sets have changed even when only one has changed, despite no changes in the DATADIGEST string.

Session Info 



      		
    

      
  





        


            

              
            

        

      


      
    








      

    



      

  
          

  

    
  

  


    

        @gfinak
gfinak




            self-assigned this


    Mar 14, 2019

  





  

  

    
  

  


    

        @gfinak
gfinak




        added
  the 

  bug

 label


    Mar 14, 2019

  





  

  

    
  

  


    

        @mariev
mariev




        added
  the 

  0.16

 label


    Mar 16, 2021

  





  


  
    
  
  

    
  slager 

      added a commit
      that referenced
      this issue

    
      May 1, 2024
    
    

      


  

    

      


  

      
        @slager
  





      

        
          bugfix for #67, fix bad test, rm purrr dependency
        


      


      

          

    
    
    

  

      


      

          

    
    

  

  

    
  



      


      
      

        
          831bcd2
        
      

    

  




    

  




  

  

    
  


  

      

    @slager
slager



    mentioned this issue
    
      May 1, 2024
    





  


  

      
   Merged

  







  




  


  
    
  
  

    
  slager 

      added a commit
      that referenced
      this issue

    
      May 1, 2024
    
    

      


  

    

      


  

      
        @slager
  





      

        
          Merge pull request #145 from ropensci/news_bugfix
        

          
            
          

      


      

          

    
    
    

  

    


      


      

          

    
    

  

  

    
  



      


      
      

        
          a924868
        
      

    

  

    

      
Bugfix for #67, fix bad test, rm purrr dependency

    




    

  




  

  

    
  

  


    

        @slager
slager




          closed this as completed


    May 1, 2024

  




  











  
  

    

      

  





  




  
  

              

  
    Sign up for free
    to join this conversation on GitHub.
    Already have an account?
    Sign in to comment


  



  






  
                  




    

  


      
  

    Assignees
  



        

      

        

  
    @gfinak
  
    gfinak
    

      








      


  


  

    Labels
  



    

      

    0.16

  

    bug









      

  

    

        

    Projects
  


        




    None yet




  



      

  

    
  

    Milestone
  


      No milestone





    
      
          


  

    
      

        
    

    Development
  




              
No branches or pull requests







    
  




        
      

    

  
      

  

    

      3 participants
    

    

        
          @mariev 
        
          @gfinak 
        
          @slager